Skip to content

Conversation

@YashK2005
Copy link
Collaborator

@YashK2005 YashK2005 commented Jan 4, 2026

Main Things

  1. Added a FRONTEND_URL env variable to remove hardcoded references to localhost:3000 in our backend.

  2. In the welcome page, there was a "You can learn more about the program here" button that before didn't take us anywhere. Now it will take us to either 'https://www.bloodcancers.ca/first-connection-peer-support-program' or 'https://www.cancersdusang.ca/programme-de-soutien-par-les-pairs-premier-contact' depending on the user's language.

  3. Added a ENV env variable that should be set to either development or production. Updated seeder so that when ENV=production, it won't seed our mock users and ranking preferences.

  4. Updated list of allowed admins to include 5 ppl from LLSC (Megan, Caroline, 3 others)

Please update your env files to have the new FRONTEND_URL and ENV environment variables.

Notion ticket link

Ticket Name

Implementation description

Steps to test

What should reviewers focus on?

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

…t in the backend, update welcome page so that it the learn more link actually takes us somehwere based on the user's language
@YashK2005 YashK2005 marked this pull request as ready for review January 4, 2026 07:26
@YashK2005 YashK2005 requested a review from ebwu95 January 7, 2026 02:52
@ebwu95 ebwu95 merged commit 52a3973 into main Jan 7, 2026
5 checks passed
@ebwu95 ebwu95 deleted the yash/welcome-page-links branch January 7, 2026 06:15
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.

3 participants