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

ui: admin message logs #2061

Merged
merged 96 commits into from
Jan 20, 2022
Merged

ui: admin message logs #2061

merged 96 commits into from
Jan 20, 2022

Conversation

Forfold
Copy link
Contributor

@Forfold Forfold commented Dec 20, 2021

This PR adds a new route to the admin pages, /admin/logs to display Outgoing Logs based on the schema changes introduced in #2052.

The first 1,000 results are being pulled in and are paginated via a load more button when scrolling to the bottom of the list for this first pass. We are also utilizing a new dependency, Fuse.js, for fuzzy searching since the data is not filtered within the GraphQL layer.

Additional work on the layout can be optimized in a future release.

It also adds the ability to use an optionally render AppLink with an icon.

Known issues / Next Up:

  • Needs test coverage
  • Discuss search keys
  • Discuss infinite loading (load size, functionality, etc)
  • Mobile optimization needed still (for filter/search area)
  • Make logs list virtualized
  • Possibly add additional filters dropdown
  • Possibly narrow card list max-width
  • Fix ISOPicker

Screenshots:

Screen Shot 2021-12-22 at 10 16 38 AM

Screen Shot 2021-12-22 at 10 16 51 AM

spencerpauly and others added 30 commits December 13, 2021 10:50
Forfold and others added 2 commits January 13, 2022 12:25
@dctalbot dctalbot self-requested a review January 20, 2022 15:59
mastercactapus
mastercactapus previously approved these changes Jan 20, 2022
dctalbot
dctalbot previously approved these changes Jan 20, 2022
@Forfold Forfold merged commit e4f77da into master Jan 20, 2022
@Forfold Forfold deleted the message-logs-frontend branch January 20, 2022 22:28
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.

4 participants