Skip to content

Conversation

@tbrand
Copy link
Contributor

@tbrand tbrand commented Apr 10, 2025

Description of Changes

Previously, interactively copying video files, but since there was a risk of triggering API Gateway timeouts, I created a job to perform the copying asynchronously.

Checklist

  • Executed npm run lint
  • Modified relevant documentation
  • Verified operation in local environment
  • Executed npm run cdk:test and if there are snapshot differences, execute npm run cdk:test:update-snapshot to update snapshots

Related Issues

#988

@tbrand tbrand changed the title [DNM] Video job async [DNM] Video job async (review after merging reel v1.1) Apr 10, 2025
@tbrand tbrand added the minor Changes equivalent to a minor version increment label Apr 16, 2025
@tbrand tbrand changed the title [DNM] Video job async (review after merging reel v1.1) Job to copy video async Apr 16, 2025
@tbrand tbrand marked this pull request as ready for review April 16, 2025 02:36
Copy link
Contributor

@wadabee wadabee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wadabee wadabee merged commit 55d652b into main Apr 16, 2025
2 checks passed
@wadabee wadabee deleted the video-job-async branch April 16, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Changes equivalent to a minor version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants