OCA / pms

Property Management System on Odoo
GNU Affero General Public License v3.0
68 stars 83 forks source link

Refactoring #81

Closed max3903 closed 2 years ago

max3903 commented 2 years ago

Is your feature request related to a problem?

pms seems to be covering a pretty large scope which makes the module more complicated to evaluate, learn, select, maintain and migrate.

Describe the solution you'd like

I would like to explore/propose splitting the module:

Describe alternatives you've considered

Additional context

My customer is renting properties to owners, furnishing them (furniture value depreciates over time so the requirement for pms_account_asset) and selling reservations on various websites.

eantones commented 2 years ago

Hello @max3903 , thank you for your comments. At this moment, the OCA PMS is focused on Hotel Management and, according to your comments and the mention of the Serpent solution, it seems you are referring to something more generic, something more Real Estate oriented such as property master and creation, property tenancy, maintenance process, commission calculation, recurring expenses, etc.. Definitely this is something we may consider in the future and any contribution on that aspect will be welcome.

Regarding the main PMS module, we totally agree with you, it's growing too much and we're gonna need to split it into several modules sooner than later. We thought about doing it taking advantage of the v15 migration but we might evaluate do do it in this very v14 as well. Lastly I'd like to let you know there's another module, it's a module based on the OCA Connector and it's used to connect to a Wubook channel manager. It's still in a PR but it's stable and already in production, we are finishing the final refactor before pushing it to the main branch.

Please let me know if you need any further information.

Thanks :-)

max3903 commented 2 years ago

Thanks for your answer @eantones.

Since you mentioned integration, I saw in some screenshots some Booking.com and Airbnb stuff. Are you integrated with Guesty (https://guesty.com)?

max3903 commented 2 years ago

@eantones Here is a 1st version of the plan to refactor:

pms_base (in #84)

Menu

Groups

Technical groups

pms_account (in #87)

pms_account_asset (in #88)

pms_contract (in #89)

pms_sale

pms_crm

pms_website

pms_website_sale

DarioLodeiros commented 2 years ago

Hi Max! Thanks for your interest and your participation. I wanted to comment on several points:

Tell me if you think it is ok to propose this refactoring for v15, and we will see how to carry it out. Thanks!

max3903 commented 2 years ago

@DarioLodeiros My project is on version 14 and I need to deliver by the end of the year.

carlosrve commented 2 years ago

Hi @max3903 ! We will be implementing PMS in a project, we are deciding right now if we migrate the code to v15 or if we stick with v14. What are your plans with this refactor?

Our timeframe will not allow us to wait or absorbe the refactor on v15 before going into production, but we could participate in the refactor if we introduce it after migrating the existing code to v15 and then proposing a PR with the respective migration scripts. In that way we could migrate the code to v15 in the next days and propose the PR.

cc @hugho-ad @julioserna

DarioLodeiros commented 2 years ago

Hello good! We are currently implementing and developing the solution in a hotel chain with more than 60 establishments, that is why we were not yet able to participate in the refactoring. But if it's okay with you, I think it's good to do a video call session to coordinate that refactoring work, at least I could tell you about known problems and the roadmap that we have. I remain at your disposal.

carlosrve commented 2 years ago

Thanks @DarioLodeiros ! OK yes. Let's wait for @max3903 response to coordinate the next step.

For us it is feasible to work after December, for the same reason, we have to deploy this into production, in v15 in the next month, but we will not be stopping there. So we could redirect some of the project resources to the refactor and cleanup of this app.

pka2why commented 2 years ago

Hello @DarioLodeiros I am looking for a solution for small hotel to work with odoo and some booking services. I see you mention there is a working solution using Wubook.
Is that a solution we can use over here in Dominican Republic ? i am really interested in this solution, please if you or anyone here could give a bit more of information regarding this idea i will really appreciate the help. Sorry if i using this tool wrong.

Thanks

max3903 commented 2 years ago

@carlosrve @DarioLodeiros My customer is live and the modules based on pms_base starts to be pretty stable now. I also have more availability to discuss. When would be a good time for you to have a call? I am in Mexico General timezone.

DarioLodeiros commented 2 years ago

Sorry I missed reading these messages. Glad to be able to have that contact to review the repository plan. @max3903 , if you can schedule a day from March 7 you can count on me, I'll send you my email dario@roomdoo.com in case you create the event for the invitation. I am in Spain, so I understand that to fit in well it must be in the morning in Mexico and in the afternoon in Spain, right? @carlosrve , do you join the session?

max3903 commented 2 years ago

@DarioLodeiros Sorry for the late answer. I sent you an invitation for Wednesday if you still want to discuss.