Skip to content

fix(#0034): add missing application link in admin-tool header#85

Merged
SigifredoChaconB merged 11 commits into
7369-admin-angular-upgradefrom
improvement/application-link-34
May 25, 2026
Merged

fix(#0034): add missing application link in admin-tool header#85
SigifredoChaconB merged 11 commits into
7369-admin-angular-upgradefrom
improvement/application-link-34

Conversation

@SigifredoChaconB

Copy link
Copy Markdown

Description

Fixes the missing Application link in the Admin-tool header.

Changes include:

  • Added Application link to HeaderComponent pointing to the main CHT application via LocationService.path
  • Updated Log out icon from fa-sign-out to fa-power-off to match the original app style
  • Added aria-hidden="true" to decorative icons in the header nav links
  • Injected LocationService into HeaderComponent to resolve the application URL
  • Added unit tests for the Application link, webAppUrl property and updated icon tests
Screenshot 2026-05-25 at 9 15 11 AM Screenshot 2026-05-25 at 9 21 22 AM

Code review checklist

  • Readable: Concise, well named, follows the style guide, documented if necessary.
  • Tested: Unit and/or e2e where appropriate
  • Internationalised: All user facing text

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

SigifredoChaconB and others added 11 commits May 5, 2026 15:33
* changes based on feedback, show errors and extract base-modal.less

* change made based on reviewer feedback
…ove setTimeout (#61)

* changes based on feedback, show errors and extract base-modal.less

* subscription to the changes service and timeouts were eliminated

* change made based on reviewer feedback
@SigifredoChaconB SigifredoChaconB merged commit 9978e6d into 7369-admin-angular-upgrade May 25, 2026
116 of 127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants