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

feat: Upload captions #483

Merged
merged 3 commits into from
Feb 24, 2025
Merged

feat: Upload captions #483

merged 3 commits into from
Feb 24, 2025

Conversation

Betree
Copy link
Member

@Betree Betree commented Nov 4, 2024

No description provided.

@Betree Betree self-assigned this Nov 4, 2024
@Betree Betree force-pushed the feat/upload-captions branch 2 times, most recently from b52eb68 to 4f6eb5d Compare February 23, 2025 23:17
@Betree Betree marked this pull request as ready for review February 23, 2025 23:17
@Betree Betree force-pushed the feat/upload-captions branch 2 times, most recently from f0748b8 to c9531b6 Compare February 23, 2025 23:35
- Trying to fetch directly from the API is too dangerous at this point, since the API is blocked
- I've chosen to iterate on a GraphQL subtitles upload API
- Trying the GraphQL mutation with Absinthe, I got a "No query document supplied" error
	-
	  ```gql
	  mutation SetVideoCaptions($captions: Upload!) {
	    setVideoCaptions(videoId: 1, captions: $captions) {
	      captions {
	       text
	      }
	    }
	  }
	  ```
- Decided to upgrade Absinthe to see how that goes
- [absinthe_ecto](https://github.com/absinthe-graphql/absinthe_ecto) is deprecated, so I'm looking to move to [dataloader](https://github.com/absinthe-graphql/dataloader)
- Updated the dependencies, now looking to migrate the actual `assoc` calls
@Betree Betree force-pushed the feat/upload-captions branch from c9531b6 to 53c9d81 Compare February 23, 2025 23:36
@Betree Betree added this pull request to the merge queue Feb 24, 2025
Merged via the queue into staging with commit dc253a0 Feb 24, 2025
6 checks passed
@Betree Betree deleted the feat/upload-captions branch February 24, 2025 22:17
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.

1 participant