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

Add database migration to enable pgcrypto #28

Merged
merged 2 commits into from
Jun 13, 2024
Merged

Conversation

rocketnova
Copy link
Contributor

Ticket

  • N/A

Changes

What was added, updated, or removed in this PR.

  • Add database migration to enable pgcrypto postgresql extension

Context for reviewers

Background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers.

Using UUIDs with Postgresql requires the Postgresql extension pgcrypto to generate random UUIDs. This PR adds a database migration to enable the extension.

Note: Depending on deployment and hosting, this may be an extension that requires elevated privileges to enable. That's the case when deploying with AWS. I will be noting that in the forthcoming PR documenting integration with the Nava Platform infrastructure template.

Testing

Provide testing instructions and evidence that the code works as expected. Include screenshots, GIF demos, shell commands or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

Before the migration:
CleanShot 2024-06-12 at 17 12 15@2x

After the migration:
CleanShot 2024-06-12 at 17 12 29@2x

After rollback:
CleanShot 2024-06-12 at 17 12 52@2x

Copy link
Contributor

@ellery-nava ellery-nava left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - seems like something we should add to the accelerator repo if we have time?

@rocketnova rocketnova merged commit c2cc7e5 into main Jun 13, 2024
5 checks passed
@rocketnova rocketnova deleted the rocket/enable_pgcrypto branch June 13, 2024 21:37
@rocketnova
Copy link
Contributor Author

LGTM - seems like something we should add to the accelerator repo if we have time?

👍 It's already in the accelerator repo. I missed it in the port to the rails template.

rocketnova added a commit that referenced this pull request Jun 25, 2024
rocketnova added a commit that referenced this pull request Jun 25, 2024
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.

2 participants