-
Notifications
You must be signed in to change notification settings - Fork 11
Quick start tweaks #980
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
Quick start tweaks #980
Conversation
✅ Deploy Preview for antenna-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Pull Request Overview
Quick start improvements to enhance the user experience during image upload workflow based on user testing feedback.
- Reordered upload dialog sections to show image selection before station selection
- Added success message and navigation options after upload completion
- Pre-selected first available station in the form
- Made capture timestamps clickable links to detail views
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| ui/src/pages/project/summary/summary.tsx | Updated state management for upload dialog visibility |
| ui/src/pages/captures/upload-images-dialog/upload-images-dialog.tsx | Reordered sections, added success screen, and pre-selected first station |
| ui/src/pages/captures/captures.tsx | Added state management for upload dialog |
| ui/src/pages/captures/capture-columns.tsx | Made timestamp column clickable with links to detail view |
| ui/src/data-services/hooks/captures/useUploadCaptures.ts | Removed unused timeout callback functionality |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
ui/src/pages/captures/upload-images-dialog/upload-images-dialog.tsx
Outdated
Show resolved
Hide resolved
|
I will go ahead and merge this. Copilot did catch one mistake I made when I changed the order of sections, nice! |

Summary
Some small tweaks after quick start user testing.
List of Changes
Related Issues
Follow up to #923
Detailed Description
How to Test the Changes
Screenshots
After upload with processing enabled:

After upload with processing disabled:
