Merge lp:~bac/launchpad/bug-422334-contact-team into lp:launchpad
Proposed by
Brad Crittenden
Status: | Merged |
---|---|
Approved by: | Curtis Hovey |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~bac/launchpad/bug-422334-contact-team |
Merge into: | lp:launchpad |
Diff against target: | None lines |
To merge this branch: | bzr merge lp:~bac/launchpad/bug-422334-contact-team |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Curtis Hovey (community) | code | Approve | |
Review via email:
|
To post a comment you must log in.
= Summary =
Bug 422334 is about the 'Contact team' link being in the team details portlet on
team-index.pt and not in the side action portlet as in person-index.pt.
== Proposed fix ==
Move the contact link to the side. Since the code from person-index.pt needs to be
duplicated in team-index.pt it was turned into a macro in person-macros.pt for re-use.
The new layout can be seen at: people. canonical. com/~bac/ bug-422334- contact- team/
http://
== Pre-implementation notes ==
IRC notes from Curtis.
== Implementation details ==
As above.
== Tests ==
None as the link is checked in existing tests but not placement.
== Demo and Q/A ==
Go to a team page and verify the placement.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: registry/ templates/ team-index. pt registry/ templates/ person- index.pt registry/ templates/ person- macros. pt
lib/lp/
lib/lp/
lib/lp/