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.

Sign up for GitHub

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

Jump to bottom

Allow users to view RFD processing jobs #125

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Allow users to view RFD processing jobs #125

wants to merge 5 commits into from

Conversation

benjaminleonard
Copy link
Collaborator

CleanShot 2025-03-19 at 14 31 23

@vercel Vercel
Copy link

vercel bot commented Mar 19, 2025

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

Name Status Preview Updated (UTC)
rfd-site ✅ Ready (Inspect) Visit Preview Mar 19, 2025 2:33pm

@david-crespo
Copy link
Contributor

Looks great. I feel like we could do better on the button label but I can’t think of anything better. Maybe a little copy at the top of the modal explaining what they are?

@benjaminleonard
Copy link
Collaborator Author

Good idea. I think I should also mention that I'm limiting the jobs in the response. In theory I could just include it all, but it feels like the user is really on interested in the most recent jobs.

@augustuswm
Copy link
Contributor

augustuswm commented Mar 19, 2025 •
edited
Loading

Awesome! Yeah, honestly job information outside of the most recent few are not terribly useful until we support viewing revisions (which needs work on the API side before we can do that).

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

@augustuswm augustuswm augustuswm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@benjaminleonard @david-crespo @augustuswm