Skip to content

Rails/Node: Installing PostgreSQL on MacOS using Postgres.app #29169

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Rails/Node: Installing PostgreSQL on MacOS using Postgres.app #29169

wants to merge 6 commits into from

Conversation

crespire
Copy link
Contributor

@crespire crespire commented Dec 9, 2024 •
edited
Loading

Because

Postgres.app makes managing and installing PSQL and its associated tooling much easier on macOS, and lets us also change versions of PSQL easily.

This PR

  • Updates instructions to use Postgres.app to install PostgreSQL on MacOS on the Rails path
  • Replicates changes to the NodeJS path lesson

This PR is in draft, as I have an outstanding question on the related issue about how we should set up local authentication.

Issue

Closes #29164

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added Content: NodeJS Involves the NodeJS course Content: Ruby on Rails Involves the Ruby on Rails course labels Dec 9, 2024
@KevinMulhern KevinMulhern self-assigned this May 3, 2025
@KevinMulhern KevinMulhern self-requested a review May 3, 2025 12:24
@KevinMulhern KevinMulhern removed their assignment May 3, 2025
KevinMulhern
KevinMulhern approved these changes May 3, 2025
View reviewed changes
Copy link
Member

@KevinMulhern KevinMulhern left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Always love to see more lines removed than added, this simplifies things a lot. Nice work @crespire 💪

Just a minor suggestion with using a warning box - but I think this is good to go once thats in place.

nodeJS/express/installing_postgresql.md

#### 3.1 PostgreSQL roles
Please note that that Postgres.app installs servers without any security configured, and will accept all local connections without a password. This is fine for our development environment, but be aware that when you move things to production, this will not be a good strategy.
Copy link
Member

Choose a reason for hiding this comment

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

Nit: Can we wrap this in a warning box - just to make sure it's harder to miss.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, @KevinMulhern ! I have additional changes to make re: peer authentication. I recall we want to stick with it, but add some notes to ensure that people are aware that's what is happening under the hood.

Should I also replicate these changes for the NodeJS path as well? I can do so in this PR as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh this is the node path file, so I'd have to replicate to the Rails path!

Copy link
Member

@KevinMulhern KevinMulhern May 7, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Thanks @crespire, feel free to send me a ping when you're happy for it to be reviewed

should I also replicate these changes for the NodeJS path as well? I can do so in this PR as well.

Yes please, these changes will be great on both paths!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KevinMulhern KevinMulhern KevinMulhern approved these changes

Assignees
No one assigned
Labels
Content: NodeJS Involves the NodeJS course Content: Ruby on Rails Involves the Ruby on Rails course
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rails: Update PSQL Install instructions on macOS
2 participants
@crespire @KevinMulhern