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

Always on tenancy #79

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ramontayag
Copy link
Contributor

@ramontayag ramontayag commented Sep 1, 2016

This is a breaking change. See notes in CHANGELOG.md.

ramontayag added a commit to bloom-solutions/plutus that referenced this pull request Jul 9, 2018
Until we merge the feature that always turns on tenancy
(mbulat#79), we must keep
the dummy Rails app's migrations disjoint from what
`rake plutus:migrations:install` actually copies over
ramontayag added a commit to bloom-solutions/plutus that referenced this pull request Jul 9, 2018
Keep tenant column:

Until we merge the feature that always turns on tenancy
(mbulat#79), we must keep
the dummy Rails app's migrations disjoint from what
`rake plutus:migrations:install` actually copies over
@ramontayag ramontayag changed the title Always on tenancy WIP: Always on tenancy Jul 9, 2018
mbulat pushed a commit that referenced this pull request Oct 31, 2019
Keep tenant column:

Until we merge the feature that always turns on tenancy
(#79), we must keep
the dummy Rails app's migrations disjoint from what
`rake plutus:migrations:install` actually copies over
So that we can generate migration files the way Rails guides suggests
This is the typical configuration of an Rails engine
Simplifies code and testing
@ramontayag ramontayag marked this pull request as draft November 16, 2020 08:39
@ramontayag ramontayag changed the title WIP: Always on tenancy Always on tenancy Nov 16, 2020
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