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.

Sign up for GitHub

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

Jump to bottom

Expose Active Storage routes #31429

Merged
merged 1 commit into from
Dec 13, 2017
Merged

Expose Active Storage routes #31429

merged 1 commit into from
Dec 13, 2017

Conversation

meinac
Copy link
Contributor

@meinac meinac commented Dec 13, 2017

Closes #31420

@rails-bot
Copy link

r? @georgeclaghorn

(@rails-bot has picked a reviewer for you, use r? to override)

@georgeclaghorn
Copy link
Contributor

There are some tests for rails routes that need to be updated.

@meinac
Copy link
Contributor Author

meinac commented Dec 13, 2017

done. let's see what the CI says.

@georgeclaghorn georgeclaghorn merged commit bf4ee05 into rails:master Dec 13, 2017
@georgeclaghorn
Copy link
Contributor

Thanks!

@GBH GBH mentioned this pull request Jan 8, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@georgeclaghorn georgeclaghorn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@meinac @rails-bot @georgeclaghorn