Skip to content

fix: resolve migration for UpdateDateColumn without ON UPDATE clause#7057

Merged
pleerock merged 2 commits intotypeorm:masterfrom
jafiot:fix6995
Jan 12, 2021
Merged

fix: resolve migration for UpdateDateColumn without ON UPDATE clause#7057
pleerock merged 2 commits intotypeorm:masterfrom
jafiot:fix6995

Conversation

@jafio
Copy link
Contributor

@jafio jafio commented Nov 11, 2020 •
edited
Loading

Closes: #6995

Description of change

When generating migration for MySQL and MariaDB, @UpdateDateColumn() now generate a default ON UPDATE clause in the up() method

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
  • Documentation has been updated to reflect this change
  • The new commits follow conventions explained in CONTRIBUTING.md

@pleerock pleerock merged commit ddd8cbc into typeorm:master Jan 12, 2021
@pleerock
Copy link
Member

pleerock commented Jan 12, 2021

Thank you for contribution! 🎉

@jafio jafio deleted the fix6995 branch January 13, 2021 03:15
This was referenced Mar 15, 2021
Merged
Closed
Closed
Merged
Closed
zshipleyTAG pushed a commit to Amherst-Development/typeorm that referenced this pull request Oct 7, 2022
@zship
Merge branch 'typeorm-0.2.30' into zs/merge-typeorm-0.2.30
56a3821 
* typeorm-0.2.30: (212 commits)
  version bump
  docs: fix javascript usage examples (typeorm#7031)
  fix: resolve migration for UpdateDateColumn without ON UPDATE clause (typeorm#7057)
  fix: Error when sorting by an embedded entity while using join and skip/take (typeorm#7082)
  fix: resolves Postgres sequence identifier length error (typeorm#7115)
  feat: closure table custom naming (typeorm#7120)
  feat: relations: Orphaned row action (typeorm#7105)
  docs: fix invalid code block in "find many options" (typeorm#7268)
  docs: Embodying the example (typeorm#7116)
  docs: document withDeleted option (typeorm#7132)
  fix: return 'null' (instead of 'undefined') on lazy relations that have no results (typeorm#7146) (typeorm#7147)
  docs: update cascade options (typeorm#7140)
  docs: add .ts to supported ormconfig formats (typeorm#7139)
  fix: improve stack traces when using persist executor (typeorm#7218)
  refactor: remove Oracle multirow insert workaround (since typeorm#6927) (typeorm#7083)
  feat: add NOWAIT and SKIP LOCKED lock support for MySQL (typeorm#7236)
  docs: update OneToMany grammar (typeorm#7252)
  feat: JavaScript file migrations output (typeorm#7253)
  docs: update Repository.ts (typeorm#7254)
  chore: update dependency cli-highlight to v2.1.10 (typeorm#7265)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Generating migrations for UpdateDateColumn should generate on update clause

2 participants

@jafio @pleerock