Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

Towards v11.2.0 #666

Merged
merged 20 commits into from
Aug 10, 2023
Merged

Towards v11.2.0 #666

merged 20 commits into from
Aug 10, 2023

Conversation

steve-chavez
Copy link
Member

Closes #644.

@laurenceisla
Copy link
Member

Since we're reverting using computed columns for disambiguation. I made two possible alternatives:

  1. Keeping the "Multiple M2O" and "Multiple O2M" examples: 25549d7
  2. Keeping the old structure using "Hint" with vews as example. But with a new title: "Multiple FK Relationships to Many Resources": a612db9

The first version is more straightforward but it's not that accurate since only the foreign key can be used for both those cases (m2o and o2m), which is simpler.

The second one is more accurate in explaining when it's needed, but not so straightforward as the previous one.

@steve-chavez
Copy link
Member Author

"Multiple FK Relationships to Many Resources"

Since we're explaining FK joins as a database concern, the term "resources" doesn't seem fit. I've removed that part and restored the previous examples plus used the hint syntax.

@steve-chavez steve-chavez marked this pull request as ready for review August 10, 2023 13:33
@steve-chavez
Copy link
Member Author

Looks good.

Will merge now despite having a broken link on https://github.com/PostgREST/postgrest/releases/tag/v11.2.0. It will be fixed once the core repo has a release.

Doing PostgREST/postgrest#2814 will prevent getting that error later.

@steve-chavez steve-chavez merged commit cacbbc3 into PostgREST:main Aug 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Move release notes pages to github releases
2 participants