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

feat(legacy): 💄 add recommendation for our AI course #478

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

BleedingDev
Copy link
Collaborator

@BleedingDev BleedingDev commented Oct 30, 2024

feat(legacy): 💄 add recommendation for our AI course

WIP: Needs changing main video

This reverts banner feature commits.

@BleedingDev BleedingDev added the enhancement New feature or request label Oct 30, 2024
@BleedingDev BleedingDev requested a review from xhemalov October 30, 2024 22:01
@BleedingDev BleedingDev self-assigned this Oct 30, 2024
Copy link

vercel bot commented Oct 30, 2024

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

Name Status Preview Comments Updated (UTC)
coursition ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 10:31am
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 10:31am

@BleedingDev BleedingDev changed the title Revert "fix(legacy): 💄 fix banner on mobile" feat(legacy): 💄 add recommendation for our AI cours Oct 30, 2024
@BleedingDev BleedingDev changed the title feat(legacy): 💄 add recommendation for our AI cours feat(legacy): 💄 add recommendation for our AI course Oct 30, 2024
@BleedingDev BleedingDev marked this pull request as ready for review October 30, 2024 23:02
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/lucide-react@0.453.0 None 0 24.6 MB ericfennis
npm/nanoid@5.0.8 None 0 10.9 kB ai
npm/next-auth@4.24.10 environment, network +14 5.9 MB balazsorban
npm/next@15.0.2 environment, filesystem, network, shell, unsafe +11 127 MB vercel-release-bot
npm/nx@20.0.3 Transitive: environment, filesystem, network, unsafe +36 13.1 MB altan-nrwl, jack-nrwl, jameshenry, ...5 more
npm/postcss@8.4.38 environment, filesystem +2 344 kB ai
npm/prisma@5.21.1 environment +1 10.9 MB prismabot
npm/react-dom@19.0.0-rc-cae764ce-20241025 environment +3 6.47 MB react-bot
npm/react-dropzone@14.2.10 None 0 443 kB rolandjitsu
npm/react-email@3.0.1 Transitive: environment, filesystem, shell +23 135 MB gabrielmfern
npm/react@19.0.0-rc-cae764ce-20241025 environment 0 239 kB react-bot
npm/resend@4.0.0 environment, network +6 1.31 MB bukinoshita
npm/server-only@0.0.1 None 0 611 B sebmarkbage
npm/sonner@1.5.0 Transitive: environment +2 240 kB emilkowalski
npm/spacetrim@0.11.60 None 0 73.1 kB hejny
npm/tailwind-merge@2.5.4 None 0 712 kB dcas
npm/tailwindcss-animate@1.0.7 None 0 18.1 kB thejameskyle
npm/tailwindcss@3.4.13 environment, filesystem Transitive: unsafe +30 8.67 MB adamwathan
npm/ts-node@10.9.2 environment, filesystem, unsafe +16 2.12 MB blakeembrey
npm/tslib@2.8.0 None 0 89.1 kB typescript-bot
npm/typescript@5.6.3 None 0 22.4 MB typescript-bot
npm/zod-form-data@2.0.2 eval 0 292 kB airjp73
npm/zod@3.23.8 None 0 667 kB colinmcd94

View full report↗︎

@BleedingDev BleedingDev force-pushed the master branch 10 times, most recently from 93e43db to 57ca4f9 Compare November 16, 2024 22:35
@BleedingDev BleedingDev force-pushed the master branch 3 times, most recently from bc26872 to e44eaff Compare November 16, 2024 22:47
@BleedingDev BleedingDev force-pushed the master branch 10 times, most recently from 5d4477b to 6a372f4 Compare November 30, 2024 18:49
@NaucMeIT NaucMeIT requested a review from Copilot February 11, 2025 13:33

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 8 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • apps/legacy_nmit/package.json: Language not supported
  • apps/legacy_nmit/pages/_document.tsx: Language not supported
  • apps/legacy_nmit/pages/chapter/[post].tsx: Evaluated as low risk
Comments suppressed due to low confidence (1)

apps/legacy_nmit/pages/dashboard.tsx:49

  • Ensure that the new behavior introduced by the MissingBanner component is covered by tests.
<MissingBanner />
@NaucMeIT NaucMeIT requested a review from Copilot February 11, 2025 18:35

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants