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

chore(templates): fix multi-tenant example and switch to postgres #10459

Closed
wants to merge 2 commits into from

Conversation

janbiasi
Copy link

@janbiasi janbiasi commented Jan 8, 2025

What?

  • Move the multi-tenant template to Postgres
  • Added missing types to the declared Hooks
  • Fixed appearing TypeScript issues after adding missing types
  • Added prettier configs to align to the other templates
  • Added constants for role, tenant role and tenant cookie name for easier adjustments
  • Updated the README.md documentation regarding migrations and role declarations.

Why?

I was looking for a multi-tenant capable CMS and had a solid experience with Payload before, so I was excited to try the multi-tenant template, unfortunately it didn't work at all with Postgres. I also came across multiple typescript errors when navigating the generated code and decided to give the debugging and bugfixing a try.

See also #9821 and the corresponding discord thread

How?

I've created my own app based on the example and adjusted everything mentioned below until it worked OOTB with the stock template.

--

Thanks for your work on the product! ❤️

@janbiasi janbiasi force-pushed the fix/multitenant-example branch 2 times, most recently from 1dd6a9a to 12da47e Compare January 8, 2025 20:53
@janbiasi janbiasi marked this pull request as ready for review January 8, 2025 21:05
@janbiasi janbiasi force-pushed the fix/multitenant-example branch 2 times, most recently from 9cb7a9d to 2080725 Compare January 9, 2025 10:24
@janbiasi janbiasi force-pushed the fix/multitenant-example branch from 2080725 to bb7568b Compare January 9, 2025 17:39
@janbiasi
Copy link
Author

janbiasi commented Jan 9, 2025

This PR can be closed in favor of #10447 if it lands in the very near future 😃

@janbiasi
Copy link
Author

Outdated because of #10447 - closing.

@janbiasi janbiasi closed this Jan 25, 2025
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