Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Oct 1, 2024. It is now read-only.

Conversation

@xenofem
Copy link
Contributor

@xenofem xenofem commented Jul 29, 2023

I'm able to upload audio files with this, but setting the artist/title isn't working properly yet, and I'm not sure why. I figured I'd create a draft PR to check whether this approach is how you want to handle things, and see if anyone else knows what's up with audio metadata.

xenofem added 2 commits July 29, 2023 16:57
For some reason, setting title/artist for audio attachments only seems
to work with the tRPC API.
@xenofem xenofem marked this pull request as ready for review July 31, 2023 20:14
@xenofem
Copy link
Contributor Author

xenofem commented Jul 31, 2023

Setting audio metadata is working properly now that I've switched to using the tRPC API for posts.attachment.start. Currently the tRPC response wrapper types just live in src/attachments.rs since that's the only place they're used for now.

@xenofem xenofem changed the title WIP: Audio attachments Audio attachments Jul 31, 2023
@iliana
Copy link
Owner

iliana commented Jul 31, 2023

nice!

@iliana iliana merged commit e0def35 into iliana:main Jul 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants