Skip to content

Conversation

@nishoof
Copy link
Member

@nishoof nishoof commented Apr 14, 2025

DON'T MERGE THIS, GURSH & I WILL MERGE ON FRIDAY

  • home page
  • details page
  • team page
    • ui
    • get team from database
    • team invites
  • riddles page
    • ui
      • description / question
      • input
      • example solution
      • place for user to enter the output they got
    • riddle list (.../riddles)
    • riddle 1 (.../riddles/1)
    • riddle 2 (.../riddles/2)
    • riddle 3 (.../riddles/3)
  • footer nav bar
  • set up prisma
    • participants
    • teams
    • team invites
  • mobile support (just for home & team pages, cant solve riddles on phone)
  • add background grid
  • favicon
  • view transitions
  • delete old platform & vercelignore (last change)

@vercel
Copy link

vercel bot commented Apr 14, 2025

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

Name Status Preview Comments Updated (UTC)
cascade ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 19, 2025 5:57am

@gurshdev gurshdev requested review from Copilot, edwardshturman, gurshdev and jetpham and removed request for edwardshturman, gurshdev and jetpham April 14, 2025 08:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 97 out of 102 changed files in this pull request and generated 1 comment.

Files not reviewed (5)
  • .vercelignore: Language not supported
  • app/globals.css: Language not supported
  • app/page.module.css: Language not supported
  • components/FooterMenu/page.module.css: Language not supported
  • old-cascade/.editorconfig: Language not supported
Comments suppressed due to low confidence (2)

components/FooterMenu/FooterMenu.tsx:3

  • [nitpick] Consider renaming or replacing the imported CSS module with one specific to FooterMenu (e.g., FooterMenu.module.css) to improve clarity and maintainability.
import styles from "./page.module.css";

old-cascade/README.md:1

  • [nitpick] Review the header text for consistent branding; consider updating 'compsigh cascade' to match the naming used in newer sections of the project if applicable.
# compsigh cascade

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 97 out of 102 changed files in this pull request and generated no comments.

Files not reviewed (5)
  • .vercelignore: Language not supported
  • app/globals.css: Language not supported
  • app/page.module.css: Language not supported
  • components/FooterMenu/FooterMenu.module.css: Language not supported
  • old-cascade/.editorconfig: Language not supported
Comments suppressed due to low confidence (1)

components/FooterMenu/FooterMenu.tsx:17

  • If 'selectedText' is defined in the imported CSS module, consider using 'styles.selectedText' instead of the string literal to ensure consistent module-based styling.
const style = href === currPath ? "selectedText" : "";

@nishoof
Copy link
Member Author

nishoof commented Apr 15, 2025

Copilot reviewed 97 out of 102 changed files in this pull request and generated no comments.

Files not reviewed (5)
Comments suppressed due to low confidence (1)
components/FooterMenu/FooterMenu.tsx:17

  • If 'selectedText' is defined in the imported CSS module, consider using 'styles.selectedText' instead of the string literal to ensure consistent module-based styling.
const style = href === currPath ? "selectedText" : "";

selectedText is defined in globals, not the imported css module.

@edwardshturman edwardshturman self-requested a review April 18, 2025 22:28
@edwardshturman edwardshturman marked this pull request as draft April 18, 2025 22:29
@edwardshturman edwardshturman linked an issue Apr 18, 2025 that may be closed by this pull request
@edwardshturman edwardshturman changed the base branch from main to stable April 19, 2025 05:52
@edwardshturman edwardshturman changed the title Make new cascade platform Redesign platform Apr 19, 2025
Base automatically changed from stable to main April 21, 2025 10:24
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