Skip to content

Conversation

@IllGive
Copy link
Contributor

@IllGive IllGive commented Aug 27, 2023

Overview of PR

Written 📝

Standards

All need to be completed before marking a pr as ready to review

  • I have reviewed my code and confirmed it doesn't cause any major security issues or create unhandled edge cases (i.e. unhandled errors)
  • I have tested my code in production and verified that it works as intended
  • I have organized my code based on our brief style guide and made sure that it's rationally componentized
  • I have confirmed the pages I edited work on light and dark mode

Brief Summary

This PR adds an acknowledgements page that lists out every package we use with its license (covering all of our bases)

Tabulated Changes

Details

Generated by AI

Walkthrough

🤖 Generated by Copilot at a19e85d

  • Ignore /pages/acknowledgements.tsx file by Prettier to speed up formatting (link)

Poem

🤖 Generated by Copilot at a19e85d

acknowledgements.tsx
Too slow for Prettier - ah!
Added to ignore

AI Summary

🤖 Generated by Copilot at a19e85d

Added /pages/acknowledgements.tsx to .prettierignore to speed up code formatting. This is part of a pull request to improve performance.

@vercel
Copy link

vercel bot commented Aug 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
coursify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 24, 2023 5:57pm

Copy link
Contributor

@Bloxs Bloxs left a comment

Choose a reason for hiding this comment

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

Should be accessible without having to log in, also there's probably a better way to make the page

Edit: Fixed being able to access without logging in, the page formatting may be fixed by my script

@Bloxs
Copy link
Contributor

Bloxs commented Aug 27, 2023

Formatting is also super broken
image

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