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 GitHub codespaces #5378

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Fix GitHub codespaces #5378

merged 1 commit into from
Dec 20, 2024

Conversation

ethax-ross
Copy link
Contributor

@ethax-ross ethax-ross commented Dec 18, 2024

Jira-3847

Context

The GitHub codespace for ECF is not building/running. We want to fix it so that we content editors can use it to make changes easily.

Changes proposed in this pull request

  • Fix GitHub codespaces

Update to bring it inline with NPQ registration codespace configuration, as this contains various fixes to get codespaces to run and the applications are very similar. Notable fixes:

  • Expose Postgres on the correct/default port
  • Setup Postgres user for vscode
  • Manually switch bundler version prior to bundle install
  • Build API guidance as part of boot.sh
  • Disable CSRF protection (for some reason it doesn't work in codespaces)
  • Boot the app with foreman so we get JS/CSS watching

Guidance to review

Switch to the branch in GitHub and boot up a new codespace, check that:

  • Rails boots/is accessible once finished.
  • API guidance works.
  • Logging in works.
  • Changing some CSS/JS then refreshing updates the page.

Update to bring it inline with NPQ registration codespace configuration, as
this contains various fixes to get codespaces to run and the applications are
very similar. Notable fixes:

- Expose Postgres on the correct/default port
- Setup Postgres user for `vscode`
- Manually switch bundler version prior to `bundle install`
- Build API guidance as part of `boot.sh`
- Disable CSRF protection (for some reason it doesn't work in codespaces)
- Boot the app with foreman so we get JS/CSS watching
Copy link

Review app deployed to https://cpd-ecf-review-5378-web.test.teacherservices.cloud

@ethax-ross ethax-ross marked this pull request as ready for review December 18, 2024 16:09
@ethax-ross ethax-ross requested a review from a team December 18, 2024 16:09
Copy link
Contributor

@leandroalemao leandroalemao left a comment

Choose a reason for hiding this comment

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

@ethax-ross Tested here and looks like it's working well. 👏 tks

Copy link
Collaborator

@cwrw cwrw left a comment

Choose a reason for hiding this comment

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

🙌

@ethax-ross ethax-ross added this pull request to the merge queue Dec 20, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Dec 20, 2024
@ethax-ross ethax-ross added this pull request to the merge queue Dec 20, 2024
Merged via the queue into main with commit 0fc3cd5 Dec 20, 2024
29 checks passed
@ethax-ross ethax-ross deleted the 3847-fix-codespaces branch December 20, 2024 10:30
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.

4 participants