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

Notes quickfix #978

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

Notes quickfix #978

wants to merge 35 commits into from

Conversation

plumshum
Copy link
Contributor

@plumshum plumshum commented Mar 4, 2025

Summary

A quick fix for the first release of notes.
Any note in saved courses cannot be edited or deleted.
image

But the note on semester view works the same per usual

CoursePlan.and.22.more.pages.-.Personal.-.Microsoft.Edge.2025-03-04.17-31-39.mp4

This pull request is a part of the Notes Feature

Remaining TODOs:

  • there seems to be an spacing issues between the courses? I thougght this was fixed in front-end fixes so I will lookover it again.

Depends on #{number of PR}

Test Plan

Notes

Blockers

  • A newly discovered dependency that hasn’t been addressed

Breaking Changes

  • Database schema change (anything that changes Firestore collection structure)
  • Other change that could cause problems (Detailed in notes)

nidhi-mylavarapu and others added 30 commits November 6, 2024 23:37
d filter for pastel note
on and option to course card menu
…ideation

Backend logic, shake animation, and frontend draft for course notes
- fixed colors
- fixed hover states hopefully?
- fixed icons
- fixed note sizes
- fixed sliding + padding between courses with notes on semester
- fixed z-indexes
- fixed colors
- fixed hover states hopefully?
- fixed icons
- fixed note sizes
- fixed sliding + padding between courses with notes on semester
- fixed z-indexes
This reverts commit 7e33104.
@plumshum plumshum requested a review from a team as a code owner March 4, 2025 22:34
@dti-github-bot
Copy link
Member

dti-github-bot commented Mar 4, 2025

[diff-counting] Significant lines: 810.

Copy link
Contributor

github-actions bot commented Mar 4, 2025

Visit the preview URL for this PR (updated for commit f3197c8):

https://cornelldti-courseplan-dev--pr978-notes-quickfix-qphhlwp5.web.app

(expires Sat, 05 Apr 2025 05:56:29 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6d4e0437c4559ed895272bbd63991394f1e0e933

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