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

[IMP] hr_recruitment: add website module as dependency for hr_recruitment #165177

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arpr-odoo
Copy link
Contributor

This commit ensures that the website module is included as a dependency for hr_recruitment. Consequently, upon installing hr_recruitment, users will also automatically gain access to the website.

task-3880378

…ment

This commit ensures that the website module is included as a dependency for
hr_recruitment. Consequently, upon installing hr_recruitment, users will
also automatically gain access to the website.

task-3880378
@robodoo
Copy link
Contributor

robodoo commented May 13, 2024

@C3POdoo C3POdoo added the RD research & development, internal work label May 13, 2024
Copy link
Contributor

@Bertrand2 Bertrand2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!
Since hr_recruitment becomes dependant on website, that means that website_hr_recruitment doesn't have any purpose anymore 🤔 So I guess than rather than be an IMP adding a dependency on hr_recruitment, this should be a [MERGE] commit to fuse both modules together in one hr_recruitment module

@arpr-odoo
Copy link
Contributor Author

oh, I guess I screwed up heavily. 😩
I'm taking it back to dev. and will finish it as soon as possible.

@Bertrand2
Copy link
Contributor

oh, I guess I screwed up heavily. 😩

It's not a screw up @arpr-odoo ! 😄 Don't worry 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants