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

Adding New Package #3

Open
Mr-Black-Dahlia opened this issue May 19, 2019 · 3 comments
Open

Adding New Package #3

Mr-Black-Dahlia opened this issue May 19, 2019 · 3 comments

Comments

@Mr-Black-Dahlia
Copy link

Thanks for the feedback earlier was massive help. I got everything working perfectly. I was able to add new tenants and domains from the panel and it worked fine.

I decided to add a new package (https://github.com/pvtl/voyager-frontend)

It gave the main admin a working frontend. But when I try to add a new tenant I get an error. I edited tenant.php and added the command to install and copied over the migrations and seeders.

Am I missing anything else for adding a package to a tenant?

@Mr-Black-Dahlia
Copy link
Author

After double checking that migrations are there. The issue seems to be this:

SQLSTATE[42S02]: Base table or view not found: 1146 Table

With the migration in the tenant folder I can create a tenant and delete them, but I can login or view them because of this sql error.

@gruz
Copy link
Owner

gruz commented May 20, 2019

I didn't try to make it work with the https://github.com/pvtl/voyager-frontend package. Thank for the hint, it may be of use for me later :-)

Some packages use auto-discovery feature of Laravel. So they can be loaded before the multi-tenancy takes the control of the process.

In such cases we should disable autodiscovery for such a package and manually register the server provider.

You can read this note to understand what I mean https://github.com/gruz/multi-tenancy-voyager-tries/wiki/Tenant-awrare-cache-note

So I can suggest something tries to look the DB table in a system DB, while should look in the tenant one.

Anyway, you are at the start of a very interesting quest - to make something work with the tenancy (-:

@Mr-Black-Dahlia
Copy link
Author

Mr-Black-Dahlia commented May 22, 2019

@gruz Thanks for the response. I will try looking into what you mentioned. I appreciate.

yes that package would be a great addition to your base code as it would give every tenant their own frontend. Which then would really make having a subdomain/fqdn shine!


So am I going in the right direction by running the install command like how you have the voyager command ran in tenancy.php then also by adding the seeds and migrations into the tenant folder?

If I can't figure it out Ill give another response. I appreciate the insight so far!

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

No branches or pull requests

2 participants