Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does anyone actually use this module? #70

Open
Adventurousdonut opened this issue Aug 4, 2022 · 12 comments
Open

Does anyone actually use this module? #70

Adventurousdonut opened this issue Aug 4, 2022 · 12 comments

Comments

@Adventurousdonut
Copy link
Contributor

I was wondering if anyone is actually using this module as i wanted to use it myself for renting out some properties but after playing around with it for a bit i saw that it has so many errors and stuff that just doesn't work properly.

Do any of you actually use it for renting out property?

@Darkjeff
Copy link
Owner

Darkjeff commented Aug 5, 2022

Yes I use since 2012 and I have two buildings and 10 renter

@Adventurousdonut
Copy link
Contributor Author

Adventurousdonut commented Aug 5, 2022

Ah yeah, thanks for the answer.

Which Dolibarr version do you use and do you expirience any "bugs"? I'm guesisng you're using the french interface.

For me idk if i did something wrong or if some stuff are just bugged.
I'm trying out the module on Dolibarr 15.0.0 and there are quite a few bugs (some minor like just visiual and some major where stuff just straigh up doesn't work for example). My dolibarr uses the german interface but naturally uses the english module language.

For example under the tab "real estate" the names on the side menu are bugged:
grafik

I've tried searching for them in the language file but they are missing so I don't know how i could possibly add the english translation, same for "MenuNewImmoCompteur" for example, it shows "DateRelever" and "Relever" (which again can't be changed to the english equivilent as its missing in the language file):
grafik

Or for some stuff I expected (idk if its intented to be like it is right now or not) the Nummer: (Prov) to change to an automatic one in renter for example but after validating and not changing anything it just stays at (Prov).

I don't know if they're all bugs or not so i didn't want to open up issues for each one as there would be quite a few. Or what is your opinion, should i open up bug reports with all the visual issues?

Thanks

@mikygee
Copy link
Contributor

mikygee commented Aug 5, 2022

Hello Adventurous Donut,

I'm using Dolibarr 15.0.2 in french,
The bugs that you are showing are quite minor and can be solved easily by adding the proper translations.
I can help in english (not in german) I already pushed an engligh update last month.

This morning my commit from july has been merged and probably created minor problems in the english version.
This is because somefields were not compatible with translation (french or english).

However, in the module I meet many bugs which are leading me to error pages.

I don't use this module yet because I don't understand how to use it properly in the one hand and because many pages in invoice don't work.
Instead I decided to help to fix bugs as you did or open bugs if I'm technically unable to.

What about you add a german translation ?

Regards

@Adventurousdonut
Copy link
Contributor Author

Adventurousdonut commented Aug 5, 2022

Hey mikygee,

Thanks you for your answer.

Yeah it's true, they are minor visual bugs. But as you also mentioned i am also having problems with the invoice tab and a few others too.

Thanks for wanting to fix the bugs. I'm gonna do some bug reports for the visuals.

Sounds good, I'm gonna try to push the german translation today.

Best regards

@mikygee
Copy link
Contributor

mikygee commented Aug 5, 2022

Hello Adventurous Donut,

I just pushed changes that should solve (in english) some of the problems you mention.
You can resync with my repository to test or wait for DarkJeff to commit the changes.
The german translations might break a little.

Regards

@Adventurousdonut
Copy link
Contributor Author

Hey mikygee,

Thanks for the changes, I'll try them out and try to unify the german translation with the french/english one.

Best regards

@Adventurousdonut
Copy link
Contributor Author

Changed the german translation to fit your format #79

@Adventurousdonut
Copy link
Contributor Author

@mikygee could you allow Issues on your fork so i can report some bugs to you directly so i dont spam this repo's issues please.
Thanks

@mikygee
Copy link
Contributor

mikygee commented Aug 8, 2022

Gutten tag,
I made the changes.
Don't open translation issues yet, please.

@mikygee
Copy link
Contributor

mikygee commented Aug 8, 2022

You can open other issues if you see any (not translation issues for the moment).

@Adventurousdonut
Copy link
Contributor Author

Bonsoir,
Alright, will do 👍.

@mikygee
Copy link
Contributor

mikygee commented Aug 25, 2022

Hello Adventurousdonut,
I guess you can add some german translations.
On my side I have pushed changes so that fields can be translated in the translation files. I have not finished though.
Take the french translation file and see what's missing in the german one.
You can open an issue on my github to follow this up.
Regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants