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

fix: MATERIALIZED VIEW is treated as a regular VIEW which causes issues on sync #7592

Merged
merged 3 commits into from
Apr 26, 2021

Conversation

AlexMesser
Copy link
Collaborator

@AlexMesser AlexMesser commented Apr 24, 2021

Description of change

Fixes #5175
Fixes #6626

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • N/A Documentation has been updated to reflect this change
  • The new commits follow conventions explained in CONTRIBUTING.md

@AlexMesser AlexMesser changed the title [WIP] bunch of VIEW issues fix: MATERIALIZED VIEW is treated as a regular VIEW which causes issues on sync Apr 26, 2021
@AlexMesser AlexMesser merged commit f85f436 into master Apr 26, 2021
@AlexMesser AlexMesser deleted the view-fix branch April 26, 2021 11:03
@cooperfrench95
Copy link

@AlexMesser Has this been released? I am still having DROP VIEW placed into my migrations, with the latest typeorm version.

@AlexMesser
Copy link
Collaborator Author

@cooperfrench95 yes, this was released in 0.2.33. Please open a new PR/issue with a reproduction example if this is still an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants