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

Create "rinsed-v1" branch #7

Merged
merged 5 commits into from
Jan 18, 2022
Merged

Create "rinsed-v1" branch #7

merged 5 commits into from
Jan 18, 2022

Conversation

jordan-brough
Copy link
Member

Based on upstream v0.16.0, and following our new forking process.

Triggered by the need to pull this in:
thoughtbot#1972

So that we can upgrade Rails.

nhippenmeyer
nhippenmeyer approved these changes Dec 9, 2021
View reviewed changes
Copy link
Member

@nhippenmeyer nhippenmeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good start. Are we thinking of moving all our changes to Administrate currently in the app into this fork? If so it will be a big lift, but something I think we should prioritize to unblock upgrading both Rails and Administrate.

@jordan-brough jordan-brough merged commit c9d156b into rinsed-v1 Jan 18, 2022
@jordan-brough jordan-brough deleted the rinsed-v1-start branch January 18, 2022 21:51
@jordan-brough jordan-brough mentioned this pull request Jan 20, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nhippenmeyer nhippenmeyer nhippenmeyer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jordan-brough @nhippenmeyer