-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/bff #358
Feature/bff #358
Conversation
Branch "feature/bff" is not allowed to deploy to production due to environment protection rules. - that's why it failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we ever going back to biofile-finder.allencell.org? I think we may no longer need to publish to that bucket (in fact, we can probably[?] delete that bucket, and just use CloudFront to redirect requests, right?).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pgarrison @aswallace @BrianWhitneyAI @yrkim98 @tyler-foster Could one of you review this PR also?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn’t seem like this PR is directly responsible, but I would prefer this change be entirely reverted if the issue I mentioned in slack below will continue.
This PR adds BFF domain S3 bucket and CloudFront distribution ID
Test:
https://github.com/AllenInstitute/biofile-finder/actions/runs/12282978093