Skip to content

graduate cleanup + qol#472

Open
denniwang wants to merge 1 commit into
mainfrom
graduate-cleanup
Open

graduate cleanup + qol#472
denniwang wants to merge 1 commit into
mainfrom
graduate-cleanup

Conversation

@denniwang
Copy link
Copy Markdown
Contributor

Pull Request

graduate authed redirect
graduate guest mode auto load plan
remove validation, sidebar cleanup, dto cleanup

Type of Change

Please tick the boxes that best match your changes.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change requires a PCP (ie changes in deps, database, infrastructure, or package exports)

Testing

Checklist

  • I have performed a self-review of my own code
  • I have commented my code where needed
  • I have made corresponding changes to the documentation
  • I have run a build for the entire monorepo
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • All commits are atomic and my branch is cleaned (no WIP commits)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
searchneu Ready Ready Preview, Comment Apr 25, 2026 10:24pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
searchneu-docs Skipped Skipped Apr 25, 2026 10:24pm

Request Review

graduate authed redirect

graduate guest mode auto load plan

remove validation, sidebar cleanup, dto cleanup

tests for dtos and requirement utils

using dal catalog

enum in dto fix + chunks fix
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

import was bugging when i tried to build locally, because different root folder, just copied and pasted the function over

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

moving the guest page to base graduate url

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

refactoring whiteboard sidebar to have a separate section component

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

code cleanup, moving fucntions to plan util file

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

support for guest plan client on base /graduate url

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removing stuff from validation folder :(

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

moved some helper function to utils file for dal activities

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

more specific types for dtos

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added schedule mutations used in baseplanclient

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the refactored helper functions used in the sidebar

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.

1 participant