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

Support Custom Postgres Schema #1818

Open
rbourdon opened this issue Oct 19, 2023 · 2 comments · May be fixed by #2540
Open

Support Custom Postgres Schema #1818

rbourdon opened this issue Oct 19, 2023 · 2 comments · May be fixed by #2540
Labels
enhancement New feature or request

Comments

@rbourdon
Copy link
Contributor

Please check if your Feature Request has not been already raised in the Discussions Tab, as we would like to reduce duplicates. If it has been already raised, simply upvote it 🔼.

Is your proposal related to a problem?

Jackson doesn't support using a postgres schema other than 'public'. In a large enterprise projects using the npm package, depending on your architecture, using public can be undesirable due to things like database roles/permissions, existing migrations, etc.

Describe the solution you'd like

Jackson should support specifying a postgres schema.

Describe alternatives you've considered

Use a distinct user for jackson and execute ALTER USER <user_name> SET search_path TO <schema>;. This can work but it would be cleaner and safer to have native support.
@rbourdon rbourdon added the enhancement New feature or request label Oct 19, 2023
@shubham-padia
Copy link

I'd like to work on this issue

@deepakprabhakara
Copy link
Member

@shubham-padia Please go ahead, you can start with looking into TypeORM's support for Postgres schema

shubham-padia added a commit to shubham-padia/jackson that referenced this issue Apr 6, 2024
@shubham-padia
Databases: Support postgres schema in dev environment.
4235278 
Fixes boxyhq#1818.
This commit adds postgres schema support to the app logic.
The dev environment uses synchronize function to create tables,
and does not run the explicit migrations. We will add schema support
for production in the next commit.
@shubham-padia shubham-padia linked a pull request Apr 6, 2024 that will close this issue
Open
8 tasks
shubham-padia added a commit to shubham-padia/jackson that referenced this issue Apr 6, 2024
@shubham-padia
Databases: Support postgres schema in dev environment.
68d44ba 
Fixes boxyhq#1818.
This commit adds postgres schema support to the app logic.
The dev environment uses synchronize function to create tables,
and does not run the explicit migrations. We will add schema support
for production in the next commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/add postgres schema support shubham-padia/jackson
3 participants
@deepakprabhakara @rbourdon @shubham-padia