-
Notifications
You must be signed in to change notification settings - Fork 9.9k
[IMP] Referrals: Update alerts #14112
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
Conversation
Hi @jero-odoo - super small doc to review when you have time. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @larm-odoo, two small issues but otherwise this is good to go
5924f4a
to
0b4d3ef
Compare
Hi @Felicious - this is ready for a final review, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @larm-odoo ! Didn't have any comments (:
Normally, I'd just merge this but since I didn't review this for TWO WEEKS 🙈 (so sorry!!), we'd probably need to rebase this with the main 18.0 branch. Can you rebase then merge? (:
@robodoo delegate+
0b4d3ef
to
ebdb242
Compare
@robodoo r+ |
Updating to current standards, adding the 'Notify Users' section.
Original task card for this PR.