-
Notifications
You must be signed in to change notification settings - Fork 318
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: Raw Event Viewer #2249
UI: Raw Event Viewer #2249
Conversation
Thanks for opening your first pull request in the Marquez project! Please check out our contributing guidelines (https://github.com/MarquezProject/marquez/blob/main/CONTRIBUTING.md). |
Great to see progress on UI changes, @tito12! Before we can merge your PR, you'll need to sign your commits (see Sign Your Work). Mind also adding UI screenshots to your PR description as well? |
Codecov Report
@@ Coverage Diff @@
## main #2249 +/- ##
=========================================
Coverage 76.47% 76.47%
Complexity 1113 1113
=========================================
Files 216 216
Lines 5203 5203
Branches 421 421
=========================================
Hits 3979 3979
Misses 752 752
Partials 472 472 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Just wanted to say, from the looks of it, this looks great! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Great! A few suggestions for you.
…te components and utils Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Great job! Congrats on your first merged pull request in the Marquez project! |
Problem
Events existed in the API but there is no way to view them through the UI.
Related to task #2166
Solution
Events page was added with ability to filter by date and expanding payload by clicking on each event.
Checklist
CHANGELOG.md
with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary).sql
database schema migration according to Flyway's naming convention (if relevant)