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

[DP-186; 200] Contributor audio upload; editor audio curation #268

Merged
merged 21 commits into from
Aug 17, 2023

Conversation

CharlieMcVicker
Copy link
Collaborator

@CharlieMcVicker CharlieMcVicker commented Apr 6, 2023

UX changes:

  • Edit word pane now has an "EditWordAudio" section
    • Users with the group "Contributors" are given the ability to upload or record audio, as well as see all audio associated with a word (contributor/ingested, curated/not curated).
    • Users with the group "Editors" are shown all audio with the ability to check/uncheck if the audio should be shown to readers (ie. if the audio should be included in the edited collection)
  • Readers may now see multiple audio recordings for a word

Backend changes:

  • Created "curateWordAudioMutation" which requires "Editors" group and updates if audio is included in the edited collection
  • Created "attachAudioToWord" mutation that ties a bit of user uploaded audio to a specific word

@CharlieMcVicker
Copy link
Collaborator Author

I have since read #212 and am convinced it will merge fine :)

@CharlieMcVicker CharlieMcVicker marked this pull request as ready for review August 11, 2023 18:20
@CharlieMcVicker CharlieMcVicker changed the title WIP: [DP-186] As a contributor, I want the ability to upload audio to DAILP [DP-186; 200] Contributor audio upload; editor audio curation Aug 11, 2023
@GracefulLemming GracefulLemming merged commit c5b00ee into main Aug 17, 2023
5 checks passed
@GracefulLemming GracefulLemming deleted the cm/2023-04-06-contributor-audio branch October 3, 2023 22:38
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