Skip to content
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

task/WG-430: implement taggit gallery navigation #335

Merged
merged 6 commits into from
Feb 25, 2025

Conversation

nathanfranklin
Copy link
Collaborator

Overview:

This implements navigating to taggit. Note taggit doesn't have project-specific routes so we have to write to local storage.

Also tweaked the switches between tabs correctly test as all components in tabs are rendered so reworked to check if active component was in active tab.

PR Status:

  • Ready.

Related Jira tickets:

Summary of Changes:

Testing Steps:

It’s slightly difficult to test in local development since we don’t currently run Taggit alongside Hazmapper.

  1. Run unit tests
  2. Set debugger before navigate(config.taggitUrl); to see that it would navigate to localhost:/taggit-local

Clicking on tab test was broken as all components rendered so it needed to be improved
to check if that components was inside active tab
Copy link
Contributor

@sophia-massie sophia-massie left a comment

Choose a reason for hiding this comment

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

Tested with your suggestions. LGTM

@nathanfranklin nathanfranklin merged commit 5c334e5 into main Feb 25, 2025
5 checks passed
@nathanfranklin nathanfranklin deleted the task/WG-430-implement-taggit-gallery-navigation branch February 25, 2025 22:49
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