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] CRM: rewrite gamification config section #9061

Closed
wants to merge 1 commit into from

Conversation

jero-odoo
Copy link
Contributor

Addressing changes from this task: https://www.odoo.com/mail/view?model=project.task&res_id=3896512&access_token=41050eb3-596d-47cc-832b-c8d047165a39

Editing config section (regarding module installation), updating two screenshots.

@jero-odoo jero-odoo self-assigned this Apr 29, 2024
@robodoo
Copy link
Collaborator

robodoo commented Apr 29, 2024

@C3POdoo C3POdoo requested a review from a team April 29, 2024 18:30
@jero-odoo
Copy link
Contributor Author

@StraubCreative This is ready for review. Thanks!

@StraubCreative
Copy link
Contributor

@jero-odoo Conflict on #9040, addressing first before we can merge here...

Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

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

Looks good @jero-odoo

I did catch a rendering issue on the bulleted list, on line 105. Will push up a fix momentarily.

Again, waiting on merge here until the FWPs from the previous PR are merged in first.

Thanks for the quick fix!

@StraubCreative StraubCreative force-pushed the 17.0-crm-gamification-config-jero branch from 6abbd27 to 205f785 Compare May 2, 2024 18:17
@StraubCreative
Copy link
Contributor

Rebased to latest on 205f785, to include previous PR updates on this page.
All checks pass ✅
..
@robodoo r+

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

Successfully merging this pull request may close these issues.

None yet

3 participants