Skip to content

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

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.

Sign up for GitHub

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

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

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

wants to merge 5 commits into from

Conversation

BleedingDev
Copy link
Collaborator

@BleedingDev BleedingDev commented Oct 30, 2024 •
edited
Loading

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
@vercel Vercel
Copy link

vercel bot commented Oct 30, 2024 •
edited
Loading

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
@socket-security Socket Security
Copy link

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

Package New capabilities Transitives Size Publisher
npm/[email protected] None 0 24.6 MB ericfennis
npm/[email protected] None 0 10.9 kB ai
npm/[email protected] environment, network +14 5.9 MB balazsorban
npm/[email protected] environment, filesystem, network, shell, unsafe +11 127 MB vercel-release-bot
npm/[email protected] Transitive: environment, filesystem, network, unsafe +36 13.1 MB altan-nrwl, jack-nrwl, jameshenry, ...5 more
npm/[email protected] environment, filesystem +2 344 kB ai
npm/[email protected] environment +1 10.9 MB prismabot
npm/[email protected] environment +3 6.47 MB react-bot
npm/[email protected] None 0 443 kB rolandjitsu
npm/[email protected] Transitive: environment, filesystem, shell +23 135 MB gabrielmfern
npm/[email protected] environment 0 239 kB react-bot
npm/[email protected] environment, network +6 1.31 MB bukinoshita
npm/[email protected] None 0 611 B sebmarkbage
npm/[email protected] Transitive: environment +2 240 kB emilkowalski
npm/[email protected] None 0 73.1 kB hejny
npm/[email protected] None 0 712 kB dcas
npm/[email protected] None 0 18.1 kB thejameskyle
npm/[email protected] environment, filesystem Transitive: unsafe +30 8.67 MB adamwathan
npm/[email protected] environment, filesystem, unsafe +16 2.12 MB blakeembrey
npm/[email protected] None 0 89.1 kB typescript-bot
npm/[email protected] None 0 22.4 MB typescript-bot
npm/[email protected] eval 0 292 kB airjp73
npm/[email protected] 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
Copilot
Copilot AI reviewed Feb 11, 2025
View reviewed changes
Copy link

@Copilot 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 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
Copilot
Copilot AI reviewed Feb 11, 2025
View reviewed changes
Copy link

@Copilot 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 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

Reviewers

Copilot code review Copilot Copilot left review comments

@xhemalov xhemalov xhemalov approved these changes

Assignees

@BleedingDev BleedingDev

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BleedingDev @xhemalov