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

Contentful CMS integration #82

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Contentful CMS integration #82

wants to merge 12 commits into from

Conversation

ChrisFong604
Copy link
Contributor

@ChrisFong604 ChrisFong604 commented Oct 20, 2024

Describe your changes

This PR integrates the Contentful CMS into the code. The official contentful Javascript client was used. Code for the join page and [jobId] dynamic route page was re-organized.

Testing steps

  • checkout to CMS
  • run pnpm install
  • update your local environment file with the the CONTENTFUL_SPACE_ID and CONTENTFUL_ACCESS_TOKEN environment variables, which can be found in the notion documentation in the technology department
  • launch the local server, and check to ensure that all of the job listings appear in the /join route, and that clicking on an individual post leads to a dynamic route with the full job description
  • Login to contentful (credentials in notion), create a new Job Listing entry, and then check to see that the entry appears on your local server.

Once testing is complete and the correct behaviour is verified, please remove any dummy data created in contentful.

Issue ticket number and link

#66

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics?

Copy link

netlify bot commented Oct 20, 2024

Deploy Preview for sfusurgedev ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/sfusurgedev/deploys/6715d601dc3fa0d10e000c7e
😎 Deploy Preview https://deploy-preview-82--sfusurgedev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ChrisFong604
Copy link
Contributor Author

Deploy preview failed due to lack of environment variables. After updating them, the deployment preview worked: https://app.netlify.com/sites/sfusurgedev/deploys/6715d601dc3fa0d10e000c7e

@ChrisFong604 ChrisFong604 changed the title Cms Contentful CMS integration Oct 21, 2024
@brendanshen24 brendanshen24 self-assigned this Oct 22, 2024
@brendanshen24 brendanshen24 marked this pull request as ready for review October 22, 2024 06:40
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