Skip to content

Conversation

annavik
Copy link
Member

@annavik annavik commented Sep 10, 2025

Summary

As the first step towards a "project quick start", we make it easier for users to upload and explore captures. We also add a couple of empty states with suggested actions, to provide more guiding.

List of Changes

  • Add empty state for project gallery
  • Automatically navigate to new project after creation
  • Add empty state for projects without captures
  • Setup new dialog for quick uploading of captures
  • Setup a dedicated view for captures
  • Update navigation (move jobs to sidebar and add captures to top bar)

Related Issues

#923

Detailed Description

How to Test the Changes

I recommend to not test this from the preview deploy because, the backend seems a bit out of date in this environment (for example captures cannot be filtered by project). Instead, test from localhost (or first deploy the backend in the preview environment to match production).

Screenshots

Updated empty state for project gallery:
Screenshot 2025-09-10 at 11 17 41

Updated empty state for project without captures:
Screenshot 2025-09-10 at 11 18 00

New dialog to upload images:
Screenshot 2025-09-10 at 11 18 19
Screenshot 2025-09-10 at 11 18 32
Screenshot 2025-09-10 at 11 18 43

Updated navigation:
Screenshot 2025-09-10 at 11 31 50

Dedicated view for captures:
Screenshot 2025-09-10 at 11 20 05

New links to captures:
Screenshot 2025-09-10 at 11 21 37
Screenshot 2025-09-10 at 11 26 35

@annavik annavik requested a review from mihow September 10, 2025 09:41
@annavik annavik changed the title Feat/image upload Make it easier for users to explore and upload captures Sep 10, 2025
Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for antenna-preview ready!

Name Link
🔨 Latest commit 74137ef
🔍 Latest deploy log https://app.netlify.com/projects/antenna-preview/deploys/68c8c9b1e4db9d0008b1014e
😎 Deploy Preview https://deploy-preview-943--antenna-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62 (🟢 up 2 from production)
Accessibility: 80 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@annavik annavik changed the title Make it easier for users to explore and upload captures Make it easier for users to upload and explore captures Sep 10, 2025
project=project,
research_site=site,
device=device,
latitude=0,
Copy link
Member Author

Choose a reason for hiding this comment

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

This is because I noticed the default project didn't pass validation and couldn't be saved.

Copy link
Collaborator

@mihow mihow left a comment

Choose a reason for hiding this comment

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

What if we keep Jobs in the menu for now until we have another prominent place for it? Everything else seems like a safe and welcome change!

/>
<Route path="summary" element={<Summary />} />
<Route path="collections" element={<Collections />} />
<Route path="collections/:id" element={<CollectionDetails />} />
Copy link
Member Author

Choose a reason for hiding this comment

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

Setup redirect route?

def get_or_create_default_device(project: "Project") -> "Device":
"""Create a default device for a project."""
device, _created = Device.objects.get_or_create(name="Default device", project=project)
device, _created = Device.objects.get_or_create(name="Default Device", project=project)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I thought you preferred the other case!

Copy link
Member Author

Choose a reason for hiding this comment

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

Haha I have no preference really, I just want things to be consistent. I noticed that for auto created entities, it seems we often use Capitalized Case (Default Station, All Images, etc.).

Copy link
Collaborator

@mihow mihow left a comment

Choose a reason for hiding this comment

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

Tested and works. What a great enhancement! Especially paired with the auto-processing. The image failure dialog worked for me as well.

I had to make 2 small fixes, but hopefully you approve, 'cause I am merging!

@mihow mihow merged commit 8cd2c9c into main Sep 16, 2025
8 checks passed
@mihow mihow deleted the feat/image-upload branch September 16, 2025 02:29
@annavik
Copy link
Member Author

annavik commented Sep 16, 2025

Tested and works. What a great enhancement! Especially paired with the auto-processing. The image failure dialog worked for me as well.

I had to make 2 small fixes, but hopefully you approve, 'cause I am merging!

Thanks for the fixes Michael! I'm excited to have this merged! :)

I noticed a few small minor I wanted to fix, I created a follow up PR: #947

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.

2 participants