Skip to content

Conversation

@miles-grant-ibigroup
Copy link
Collaborator

@miles-grant-ibigroup miles-grant-ibigroup commented Aug 18, 2021

This PR adds search and filter capabilities to the route viewer. It also moves the route viewer to the Navbar to make it easier to find.

@miles-grant-ibigroup miles-grant-ibigroup added the WIP Work in progress label Aug 18, 2021
@miles-grant-ibigroup miles-grant-ibigroup self-assigned this Aug 18, 2021
@miles-grant-ibigroup miles-grant-ibigroup marked this pull request as ready for review August 19, 2021 11:57
@miles-grant-ibigroup miles-grant-ibigroup removed the WIP Work in progress label Aug 19, 2021
Copy link
Member

@landonreed landonreed left a comment

Choose a reason for hiding this comment

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

Love this! Just a few changes. Also, do we have a final reviewer? Could you propose one if not?

@miles-grant-ibigroup
Copy link
Collaborator Author

miles-grant-ibigroup commented Aug 20, 2021

Everything addressed! With the redux I did my best to follow project conventions, but am very open to restructuring things to match the rest of the project. Also I'd like to propose @binh-dam-ibigroup as the second reviewer as we're adding React-intl stuff

@binh-dam-ibigroup
Copy link
Collaborator

Thanks for the translations and notes! I can't respond to some of the comments, but most have been addressed in 3bcaf13 aside from:

Okay, thanks for clarifying.

Copy link
Collaborator

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Two more formatting things, but otherwise great work, thanks for the changes!

@miles-grant-ibigroup miles-grant-ibigroup added the BLOCKED Blocked (waiting on another PR to be merged) label Sep 14, 2021
@miles-grant-ibigroup
Copy link
Collaborator Author

Thanks Binh! This is now blocked waiting for piggyback PR #441 to be merged

@miles-grant-ibigroup miles-grant-ibigroup removed the BLOCKED Blocked (waiting on another PR to be merged) label Sep 20, 2021
@miles-grant-ibigroup miles-grant-ibigroup merged commit dab6175 into dev Sep 20, 2021
@miles-grant-ibigroup miles-grant-ibigroup deleted the route-viewer-filter-and-search branch September 20, 2021 17:24
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2021

🎉 This PR is included in version 3.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants