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

Format codebase and add support for multi-tenancy .. #7

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

dlobo
Copy link
Contributor

@dlobo dlobo commented Sep 18, 2020

Needed to fork the repo again to add one more feature. We needed support to run the same migration again for a different tenant. This is especially important in the initial db seeding phase (not so sure if needed for later migrations, but we'll cross that bridge when we get there). Added a tenant column to the schema_seeds table to make this possible

also formatted the code using the built in formatter

if you are open to this being merged, I'll take a closer look at the PR, delete most of the commented code and do some more cleanup

@dlobo
Copy link
Contributor Author

dlobo commented Sep 28, 2020

wondering if you have any thoughts or comments on these changes?

@dlobo
Copy link
Contributor Author

dlobo commented Oct 31, 2020

Can you let me know either way. Working towards another version of the package that we'd like to publish on hexdocs.pm

lobo

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