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: Add serverpod migrations table. #35

Closed
wants to merge 8 commits into from

Conversation

SandPod
Copy link
Owner

@SandPod SandPod commented Nov 14, 2023

Changes

  • Defines a Serverpod table for the serverpod_migrations table allowing us to use our deserialization and query support.
  • Removes the unsafe sql queries used to create and deserialize the serverpod_migrations table.

Pre-launch Checklist

  • I read the Contribute page and followed the process outlined there for submitting PRs.
  • This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
  • I read and followed the Dart Style Guide and formatted the code with dart format.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
  • I added new tests to check the change I am making.
  • All existing and new tests are passing.
  • Any breaking changes are documented below.

If you need help, consider asking for advice on the discussion board.

Breaking changes

If you have done any breaking changes, make sure to outline them here, so that they can be included in the notes for the next release.

@SandPod SandPod force-pushed the add-serverpod-migrations-table branch 2 times, most recently from 9fb6ced to 5a86379 Compare November 15, 2023 09:29
@SandPod SandPod force-pushed the add-serverpod-migrations-table branch from 31a43cc to be5f45e Compare November 15, 2023 10:26
@SandPod SandPod closed this Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant