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

Server Feedback Updates #44

Merged
merged 3 commits into from
Feb 21, 2024
Merged

Server Feedback Updates #44

merged 3 commits into from
Feb 21, 2024

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Feb 21, 2024

resolves #43
resolves #39
resolves #37
resolves #35

  • Fixes swapping between sequence and pulse and annotation editor by making so when you click on a different tab it resets the currently selected annotation to null
  • Swapped the pulse/sequence order and made it default to pulse on launch
  • Added a simple icon to delete owned recordings.
  • Updated the upload process to kick of a celery task to generate the spectrogram. In the get /recordings it now lists if the user has a spectrogram or not. If they don't have a spectrogram a warning icon and spin bar comes up where it reloads the list every 5 seconds to see if the spectrogram is loaded. If so it stops the refreshing and allows a user to click on the field.
  • Made it so the public list will filter out recordings that don't have spectrograms yet.
  • Updated the login/logout text so it reflects the proper state when logged in or out
feedbackPR.mp4

@BryonLewis BryonLewis merged commit 9ad5f94 into main Feb 21, 2024
6 checks passed
@BryonLewis BryonLewis deleted the feedback-updates branch February 22, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant