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

Upload Capybara screenshots to S3 in the CI environment. #3169

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

fonglh
Copy link
Member

@fonglh fonglh commented Oct 25, 2018

PRs from other repositories cannot read the variables defined in the repository settings. If there's a problematic PR, its branch will need to be created here instead of on our usual fork https://github.com/coursemology-collab/coursemology2.

Documentation for setting up and configuring the bucket is available https://github.com/Coursemology/coursemology2/wiki/Sending-Capybara-Screenshots-to-S3

@ghost ghost assigned fonglh Oct 25, 2018
@ghost ghost added the Needs Review label Oct 25, 2018
@fonglh fonglh force-pushed the fonglh/capybara-screenshot branch 4 times, most recently from 1fd557e to 3fa1120 Compare November 1, 2018 07:36
@fonglh fonglh added the Blocked label Nov 12, 2018
@fonglh fonglh force-pushed the fonglh/capybara-screenshot branch from 3fa1120 to 6465818 Compare January 9, 2019 06:39
@ghost ghost added the Needs Review label Jan 9, 2019
@fonglh fonglh removed the Blocked label Jan 9, 2019
@fonglh fonglh requested a review from trangmei January 9, 2019 06:47
@fonglh fonglh merged commit cfb6eb9 into master Jan 10, 2019
@ghost ghost removed the Needs Review label Jan 10, 2019
@fonglh fonglh deleted the fonglh/capybara-screenshot branch January 10, 2019 09:29
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.

2 participants