Closed
Bug 875728
Opened 11 years ago
Closed 11 years ago
Need hook for links in common-links
Categories
(Bugzilla :: Bugzilla-General, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 5.0
People
(Reporter: aliustek, Assigned: aliustek)
Details
Attachments
(1 file)
502 bytes,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
common-links need a hook to provide extensions a way to insert links MyDashboard on bmo uses "[% Hook.process('action-links') %]" link on bmo common-links. This hook could be moved to mainstream bugzilla code
Assignee: general → aliustek
Attachment #753735 -
Flags: review?(LpSolit)
Comment 2•11 years ago
|
||
Comment on attachment 753735 [details] [diff] [review] action-links hook Review of attachment 753735 [details] [diff] [review]: ----------------------------------------------------------------- r=dkl. We also have this hook in the BMO code already to support the MyDashboard extension so I am sure it works well. This one and the others for the saved searches, I meant to upstream those but never got to it so you beat me to it :)
Attachment #753735 -
Flags: review+
Updated•11 years ago
|
Flags: approval?
Comment 3•11 years ago
|
||
Is this hook really needed in addition to the existing "link-row" hook in the same template? Is the intention to have more actions on the same line as other actions?
Target Milestone: --- → Bugzilla 5.0
Comment 4•11 years ago
|
||
(In reply to Frédéric Buclin from comment #3) > Is this hook really needed in addition to the existing "link-row" hook in > the same template? Is the intention to have more actions on the same line as > other actions? Given that BMO use it, and it could be useful to other installations that also have extensions that might modify action links, I'm happy to a+ this for upstream.
Status: NEW → ASSIGNED
Flags: approval? → approval+
Updated•11 years ago
|
Attachment #753735 -
Flags: review?(LpSolit)
Comment 5•11 years ago
|
||
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bugzilla/trunk modified template/en/default/global/common-links.html.tmpl Committed revision 8663.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•