Skip to content

[17.0][ADD] mail_notification_link #134

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.

Sign up for GitHub

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

Jump to bottom
Merged
OCA-git-bot merged 1 commit into from
Jan 14, 2026
Merged

[17.0][ADD] mail_notification_link #134

OCA-git-bot merged 1 commit into from
Jan 14, 2026

Conversation

@AndreuOForgeFlow
Copy link

@AndreuOForgeFlow AndreuOForgeFlow commented Jan 9, 2026

This module makes the document name in the messaging menu notifications clickable, allowing users to navigate directly to the document (e.g., Sale Order, Purchase Order, etc.) by clicking on its name.

When clicking the display name of the document:
image

It navigates directly to the document:
image

@AndreuOForgeFlow AndreuOForgeFlow mentioned this pull request Jan 9, 2026
Closed
@AndreuOForgeFlow
Copy link
Author

AndreuOForgeFlow commented Jan 9, 2026

Moved from:

@GuillemCForgeFlow @hoangtrann @pedrobaeza

GuillemCForgeFlow
GuillemCForgeFlow approved these changes Jan 9, 2026
View reviewed changes
Copy link
Contributor

@GuillemCForgeFlow GuillemCForgeFlow left a comment

Choose a reason for hiding this comment

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

code and functional lgtm 👍🏿

hoangtrann
hoangtrann approved these changes Jan 9, 2026
View reviewed changes
Copy link

@hoangtrann hoangtrann left a comment

Choose a reason for hiding this comment

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

Code LGTM

AaronHForgeFlow
AaronHForgeFlow approved these changes Jan 12, 2026
View reviewed changes
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

👍 Functional review

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 14, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

2 similar comments
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 14, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 14, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AndreuOForgeFlow
Copy link
Author

AndreuOForgeFlow commented Jan 14, 2026

@pedrobaeza, perhaps we can merge this one?

@pedrobaeza pedrobaeza added this to the 17.0 milestone Jan 14, 2026
@pedrobaeza
Copy link
Member

pedrobaeza commented Jan 14, 2026

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 14, 2026

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-134-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 27ee9ad into OCA:17.0 Jan 14, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Jan 14, 2026

Congratulations, your PR was merged at 1eec050. Thanks a lot for contributing to OCA. ❤️

@AndreuOForgeFlow AndreuOForgeFlow deleted the 17.0-add-mail_notification_link branch January 14, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@AaronHForgeFlow AaronHForgeFlow AaronHForgeFlow approved these changes

@hoangtrann hoangtrann hoangtrann approved these changes

@GuillemCForgeFlow GuillemCForgeFlow GuillemCForgeFlow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 ready to merge

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

6 participants

@AndreuOForgeFlow @OCA-git-bot @pedrobaeza @AaronHForgeFlow @hoangtrann @GuillemCForgeFlow