Skip to content

Purva/calendar absence updating#84

Merged
ChinemeremChigbo merged 16 commits intomainfrom
purva/calendar-absence-updating
Apr 11, 2025
Merged

Purva/calendar absence updating#84
ChinemeremChigbo merged 16 commits intomainfrom
purva/calendar-absence-updating

Conversation

@dagapurva3
Copy link
Contributor

Notion Ticket

Calendar Absence Updating](https://www.notion.so/uwblueprintexecs/Calendar-Absence-Updating-6b89a97363a74b13bde8b7cd1eab1529?pvs=4)

Summary & Review Focus

  1. Added edit absence feature by modifying the existing declare absence modal
  2. Added default input for existing absence in the input form

Testing Instructions

  1. pull the branch and open /calendar url
  2. Click on any existing absence and then see the default values getting filled for the existing absence
  3. Then change the values prefilled in the input form and update it

Checklist

  • PR title is descriptive and in imperative tense
  • Commit messages are descriptive, atomic, and follow best practices
  • Linter(s) have been run
  • Requested reviews from the PL and relevant team members

@vercel
Copy link

vercel bot commented Mar 12, 2025

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

Name Status Preview Comments Updated (UTC)
sistema ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 11, 2025 10:22am

Copy link
Member

@ChinemeremChigbo ChinemeremChigbo left a comment

Choose a reason for hiding this comment

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

Please address failing deployment

@ChinemeremChigbo ChinemeremChigbo force-pushed the purva/calendar-absence-updating branch from 984cd29 to ff33aed Compare April 11, 2025 10:20
@ChinemeremChigbo ChinemeremChigbo self-requested a review April 11, 2025 10:24
@ChinemeremChigbo ChinemeremChigbo merged commit 85a6354 into main Apr 11, 2025
3 checks passed
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.

2 participants