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

Update MediaUploader ref to master (adds a merge commit only) #30208

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

dannyroberts
Copy link
Member

Summary

I had made dimagi/MediaUploader#29 along with #29489.

Product Description

None

Safety Assurance

  • Risk label is set correctly
  • All migrations are backwards compatible and won't block deploy
  • The set of people pinged as reviewers is appropriate for the level of risk of the change
  • If QA is part of the safety story, the "Awaiting QA" label is used
  • I have confidence that this PR will not introduce a regression for the reasons below

Automated test coverage

There is no code change

QA Plan

There is no code change

Safety story

There is no code change. The submodule ref is being updated to include one more commit which is a merge commit (merging in the PR) that has no code changes in itself.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

@dannyroberts dannyroberts added the product/invisible Change has no end-user visible impact label Aug 5, 2021
@dannyroberts dannyroberts merged commit c3c2500 into master Aug 5, 2021
@dannyroberts dannyroberts deleted the dmr/update-MediaUploader-ref branch August 5, 2021 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product/invisible Change has no end-user visible impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants