Skip to content

Update all packages #766

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
Merged
TyHil merged 55 commits into from
Jul 1, 2025
Merged

Update all packages #766

TyHil merged 55 commits into from
Jul 1, 2025

Conversation

@TyHil
Copy link
Member

@TyHil TyHil commented Jun 4, 2025

No description provided.

Aron Petkovski and others added 30 commits November 13, 2023 18:39
final fix, please
4f0ad83
@TyHil
Update links
3052744
@TyHil
Reroute homepage
ba7835d
@TyHil
Lint
9962681
@TyHil
Remove husky
e284c80
@TyHil
Remove .vscode
9f245c0
@TyHil
Keep .vscode removed
64598c1
@TyHil
Reduce warnings
a6ea584
@TyHil
useEffect fix
cf5d8ba
@TyHil
Fix typo
5a07b9e
@TyHil
Import fix
73c156d
@TyHil
Update link in vercel preview comment
cb0963b
@TyHil
Merge pull request #763 from UTDNebula/update-links
8dace79 
Update main page and warnings
@TyHil
Remove unused packages
9372965
@TyHil
React 18
efd5337
@TyHil
Update a lotta packages
7b95c86
@TyHil
Some more packages
274b299
@TyHil
React types
453c2be
@TyHil
trpc
8e5d609
@TyHil
Lint
633d20d
@TyHil
that damn eslint rule that changes only on tests
0341172
@TyHil
Node v22
1d0392f
@TyHil
pdfjs fix
ec1226d
@TyHil
Cypress test fix
979e319
@TyHil
Remove console.logs
e5acd8b
@TyHil
More cross-env
f884f9f
@TyHil
Update storybook
45c26bb
@TyHil
Change eslint import sorter
887f28c
@TyHil
eslint use eslint:recommended
b0b8e22
@TyHil
trpc
44afd7b
TyHil added 15 commits June 3, 2025 15:08
@TyHil
cypress fix
acbecb7 
hopefully will give the validator more time to spin up
@TyHil
Merge pull request #764 from UTDNebula/packages
e5c8186 
Update packages
@TyHil
Merge branch 'develop' into new-tab-branch
0551707
@TyHil
Change link new tab method
a99531d
@TyHil
Fix button cursor pointers
9df86d8
@TyHil
Merge pull request #700 from UTDNebula/new-tab-branch
87588a7 
Side-bar opens a New tab
@TyHil
Fix redirect
a2c45a7
@TyHil
Sitemap
2a6eea6
@TyHil
Inter
0ba40cd
@TyHil
Init
ba44358 
py -3.11 -m pip install -r requirements.txt
py -3.11 -m pip freeze > requirements.txt
@TyHil
update all
3248fe4 
py -3.11 -m pip install --upgrade absl-py async-timeout attrs beautifulsoup4 black certifi charset-normalizer check-jsonschema click Deprecated exceptiongroup Flask Flask-Cors Flask-Limiter gunicorn idna iniconfig itsdangerous Jinja2 jira jsonschema limits lxml MarkupSafe mypy mypy-extensions numpy ortools packaging pathspec pip platformdirs pluggy protobuf pydantic Pygments pyparsing pyrsistent pytest python-dotenv redis requests rich ruamel.yaml ruamel.yaml.clib setuptools six tomli types-beautifulsoup4 types-Flask-Cors types-jsonschema types-requests typing_extensions urllib3 Werkzeug wrapt
py -3.11 -m pip freeze > requirements.txt
@TyHil
Fix type errors
bf338af
@TyHil
Run black formatter
c17c011
@TyHil
Formatter github action
3785714
@TyHil
Merge pull request #765 from UTDNebula/python-packages
2544ea7 
Update validator packages
@TyHil TyHil requested review from a team and akevinge as code owners June 4, 2025 04:31
@TyHil
Copy link
Member Author

TyHil commented Jun 4, 2025

@AbhiramTadepalli and @nl32 would yall mind doing some testing to make sure nothing is broken on https://dev.planner.utdnebula.com/auth/login that isnt already broken on https://planner.utdnebula.com/auth/login

@TyHil TyHil requested review from AbhiramTadepalli and nl32 June 4, 2025 04:34
@AbhiramTadepalli
Copy link

AbhiramTadepalli commented Jun 29, 2025 •
edited by TyHil
Loading

  • Typed an email w/o acct. Followed the link from email. Created acct and filled in major and clicked the button but it basically refreshed and had to re-enter
  • Report a Bug visually interferes with the status toast messages (also right-sidebar text that comes to the bottom)
  • After uploading a transcript and clicking next, it goes to the previous instruction card while loading (enter major and name)
  • When the top card has a missing prereq, hovering for the description becomes hard and show more is not clickable
Probably unrelated bugs (I can't test in prod) - [ ] Adding Comms courses (CS degree). I added one, then deleted it. Still shows a check mark in the Right sidebar
Bugs also in prod Planner
  • 7th tutorial highlights "Sort By" but shows the text for settings ("This is where you can edit your semesters")
  • Allows me to add the same course multiple times (in diff sems), is that intended?
  • Selected a color for a course: the height of the color is less than the height of the card
  • I can spam the back button from a plan and it restarts the navigation back
  • Imported transcript (I have AP credits. That section of Transfer Credits sats "NaN Credits taken")
  • Everything page navigation is so slow
  • https://dev.planner.utdnebula.com/app/onboarding can be accessed in incognito
More so feature requests
  • Allow adding double majors / minors / search other courses (can display a warning tho)
  • Grad courses can't be added
  • CS Planner Template uses cs 1336 (outdated) -- the template should be somehow up-to-date with coursebook

AbhiramTadepalli
AbhiramTadepalli approved these changes Jun 29, 2025
View reviewed changes
Copy link

@AbhiramTadepalli AbhiramTadepalli left a comment

Choose a reason for hiding this comment

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

didn't look at the code, should I?

@TyHil
Copy link
Member Author

TyHil commented Jun 29, 2025

No I don't think you need to. Are the first 6 confirmed in dev but not prod?

@AbhiramTadepalli
Copy link

AbhiramTadepalli commented Jun 30, 2025

Are the first 6 confirmed in dev but not prod?

narrowed it to 4 I know work in prod that don't in dev

@TyHil TyHil mentioned this pull request Jul 1, 2025
Merged
1 task
@TyHil
Copy link
Member Author

TyHil commented Jul 1, 2025

Fixed one and made issues for the rest.

Thank you so much for reviewing the giant PR btw

@TyHil TyHil merged commit e4b80d3 into main Jul 1, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AbhiramTadepalli AbhiramTadepalli AbhiramTadepalli approved these changes

@akevinge akevinge Awaiting requested review from akevinge

@nl32 nl32 Awaiting requested review from nl32

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TyHil @AbhiramTadepalli