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

#153 Add application summary #160

Merged
merged 19 commits into from
May 2, 2024
Merged

Conversation

nevendyulgerov
Copy link
Contributor

@nevendyulgerov nevendyulgerov commented Apr 15, 2024

I tweaked /applications/{address} route to display the application summary information with 4 cards for inputs, notices, vouchers and reports. Below the cards, I added the 6 latest inputs table.

Copy link

vercel bot commented Apr 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rollups-explorer-mainnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2024 8:03pm
rollups-explorer-sepolia ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2024 8:03pm
rollups-explorer-workshop ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2024 8:03pm

@nevendyulgerov nevendyulgerov added Type: Feature Add a new feature to the system apps: web labels Apr 15, 2024
@nevendyulgerov nevendyulgerov linked an issue Apr 15, 2024 that may be closed by this pull request
13 tasks
@nevendyulgerov nevendyulgerov force-pushed the feature/153-add-application-summary branch from 48decb7 to 6a1fa5a Compare April 24, 2024 12:25
@nevendyulgerov nevendyulgerov marked this pull request as ready for review April 25, 2024 08:27
Copy link
Collaborator

@brunomenezes brunomenezes left a comment

Choose a reason for hiding this comment

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

@nevendyulgerov, I did some testing, and a couple of things need to be fixed. The summary and latest inputs cards are unfiltered. They show the total inputs indexed and the latest inputs sent to any application. This is a summary page for the application you are checking so filtering by its address is necessary.

@brunomenezes
Copy link
Collaborator

Good one. Just need to rebase the branch.

@brunomenezes brunomenezes merged commit 651589b into main May 2, 2024
8 checks passed
@brunomenezes brunomenezes deleted the feature/153-add-application-summary branch May 2, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apps: web Type: Feature Add a new feature to the system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Application summary under /applications/<address>
2 participants