Skip to content

Students archive page #217

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Students archive page #217

wants to merge 1 commit into from

Conversation

heyitsOK
Copy link
Member

@heyitsOK heyitsOK commented Mar 18, 2025

Description of Changes

  • Added /students/archive page
  • Moved common JSX into students layout.tsx
  • Created getCurrentStudents and getPastStudents functions so we don't have to fetch all student data to render only current/past students

Related Issues

Checklist

  • MR title is meaningful and accurate
  • This MR has an associated GitHub Issues ticket
  • Code quality check has been run npm run quality
  • Preview deployment has passed and looks as expected
  • I promise that my commit message for this MR will be clear, meaningful,
    and useful to others. I will ensure this by editing the final commit message
    in GitHub prior to merging

If a checklist item is completed for this MR, place an x inside of the square
brackets for that item. If a checklist item is not applicable for this MR, please
note that by wrapping that line with ~ characters, like this.

Make sure you squash your commits before merging!

Copy link

netlify bot commented Mar 18, 2025

Deploy Preview for cublueprint ready!

Name Link
🔨 Latest commit 94eac5a
🔍 Latest deploy log https://app.netlify.com/sites/cublueprint/deploys/67d9cd7e31c57600086c7161
😎 Deploy Preview https://deploy-preview-217--cublueprint.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.

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.

Remove previous members from main students page
1 participant