Skip to content

Commit 30d9af6

Browse files
Fix audio bugs 02202026 (#668)
* Add editors permissions for attaching word audio * add https to audio urls that do not yet have them --------- Co-authored-by: Naomi Trevino <naomi.e.trevino@gmail.com>
1 parent 823fb4a commit 30d9af6

2 files changed

Lines changed: 12 additions & 2 deletions

File tree

graphql/src/query.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -702,7 +702,9 @@ impl Mutation {
702702

703703
/// Attach audio that has already been uploaded to S3 to a particular word
704704
/// Assumes user requesting mutation recoreded the audio
705-
#[graphql(guard = "GroupGuard::new(UserGroup::Contributors)")]
705+
#[graphql(
706+
guard = "GroupGuard::new(UserGroup::Contributors).or(GroupGuard::new(UserGroup::Editors))"
707+
)]
706708
async fn attach_audio_to_word(
707709
&self,
708710
context: &Context<'_>,

types/src/database_sql.rs

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -963,10 +963,18 @@ impl Database {
963963
upload: &AttachAudioToWordInput,
964964
contributor_id: &Uuid,
965965
) -> Result<Uuid> {
966+
// TODO adopt URL type
967+
let is_url = |s: &String| s.starts_with("https://");
968+
969+
let url = if is_url(&upload.contributor_audio_url) {
970+
upload.contributor_audio_url.clone()
971+
} else {
972+
"https://".to_owned() + &upload.contributor_audio_url
973+
};
966974
let media_slice_id = query_file_scalar!(
967975
"queries/attach_audio_to_word.sql",
968976
contributor_id,
969-
upload.contributor_audio_url as _,
977+
url as _,
970978
0,
971979
0,
972980
upload.word_id

0 commit comments

Comments
 (0)