8000 Kaizen by soedirgo · Pull Request #99 · supabase/postgres-meta · GitHub
[go: up one dir, main page]

Skip to content

Kaizen #99

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

Merged
merged 5 commits into from
Feb 28, 2021
Merged

Kaizen #99

merged 5 commits into from
Feb 28, 2021

Conversation

soedirgo
Copy link
Member

What kind of change does this PR introduce?

Bug fix + some cleanup.

What is the new behavior?

  • Remove docs: these are pretty outdated, we can add back OAS stuff later when we need it.
  • Remove unused stuff from src/server: these are stubs for using the new lib, but are currently unused.
  • Move old stuff to src/old: makes it clear which parts are the new lib stuff and which are the old routes.
  • Pluralize PostgresMeta objects (PostgresMetaColumns, etc.)
  • Fix /publications: forgot to update the logic & tests in fix(publications): disambiguate tables' schemas #98. 😅 Probably better to specify tables using OIDs when creating publications, but this'll do for now.

These are pretty outdated, we can add back OAS stuff later when we need it.
Makes it clear which parts are the new lib stuff and which are the old routes.
Probably better to specify `tables` using OIDs when creating
publications, but this'll do for now.
@soedirgo soedirgo requested a review from kiwicopple February 28, 2021 02:44
@kiwicopple
Copy link
Member

image

will have to trust you on this one 😅

@kiwicopple kiwicopple merged commit f35c38f into develop Feb 28, 2021
@github-actions
Copy link

🎉 This PR is included in version 0.15.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

@soedirgo soedirgo deleted the chore/kaizen branch June 21, 2021 06:48
avallete pushed a commit that referenced this pull request May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0