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

chore(storybook): Added publishing of PR storybook to S3 bucket #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

majames
Copy link
Collaborator

@majames majames commented Jun 3, 2017

📖 (allthereviews)

Need to release PR to check this is working. @zzarcon does devlucky have a slack channel? If so, would be good to publish a link to s3 bucket PRs to slack channel in the future 😄

@majames majames force-pushed the chore/publish-storybook-on-pr branch 2 times, most recently from 211b716 to 8864428 Compare June 3, 2017 04:20
@majames
Copy link
Collaborator Author

majames commented Jun 3, 2017

😞 turns out Travis doesn't allow you to deploy on PR: travis-ci/travis-ci#7338 (comment)

Will have to look at using the aws cli instead.

@majames majames force-pushed the chore/publish-storybook-on-pr branch 3 times, most recently from fde5fa5 to 87e69e0 Compare June 4, 2017 02:30
@majames majames force-pushed the chore/publish-storybook-on-pr branch from 87e69e0 to 999b0bd Compare June 4, 2017 02:34
@majames
Copy link
Collaborator Author

majames commented Jun 4, 2017

Alright moved deploy config from travis-ci yaml config to a build script so it's working now 😄

@majames majames requested a review from zzarcon June 4, 2017 02:41
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