RolnickLab / ami-platform

GNU General Public License v3.0
8 stars 2 forks source link

Fix various onboarding issues #366

Closed mihow closed 4 months ago

mihow commented 4 months ago

Users have been unable to create a new Deployment on their own. This is mostly because a Device, Research Site and Data Source need to be created first before creating a Deployment. Those related models are not actually required, but the form was requiring something to be selected. This PR makes several updates to address this issue.

Other changes:

netlify[bot] commented 4 months ago

Deploy Preview for ami-web ready!

Name Link
Latest commit 73eb7ee5df6764c7e7458258173640bb8d32341a
Latest deploy log https://app.netlify.com/sites/ami-web/deploys/65f09d1a489c0b0008dfcc7a
Deploy Preview https://deploy-preview-366--ami-web.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 75
Accessibility: 95
Best Practices: 92
SEO: 92
PWA: 70
View the detailed breakdown and full score reports

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

netlify[bot] commented 4 months ago

Deploy Preview for ami-storybook ready!

Name Link
Latest commit 73eb7ee5df6764c7e7458258173640bb8d32341a
Latest deploy log https://app.netlify.com/sites/ami-storybook/deploys/65f09d1add3d53000855b592
Deploy Preview https://deploy-preview-366--ami-storybook.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

mihow commented 4 months ago

Hi @jonbeu this should fix the issue you faced when creating a new deployment. The site, device & data source are no-longer required -- but also a default device & site are created for each project now. I am anxious to see some test images from the AMI-Mini! -Michael