Skip to content

Conversation

@miretskyd
Copy link
Member

@miretskyd miretskyd commented Nov 10, 2025

Note

Always show a delete button for each reminder and move the add-reminder button outside the list for consistent access.

  • Frontend
    • Calendar (client/src/routes/calendar/+page.svelte)
      • Reminder UI: always render per-item delete button (remove length/last-item conditions).
      • Add-reminder button moved outside the loop to add new reminders consistently.

Written by Cursor Bugbot for commit 2aed3ea. This will update automatically on new commits. Configure here.

@miretskyd miretskyd self-assigned this Nov 10, 2025
@miretskyd miretskyd added the enhancement New feature or request label Nov 10, 2025
@miretskyd miretskyd changed the title Notification deletion Feat/notification deletion Nov 11, 2025
@Cattn Cattn changed the title Feat/notification deletion Feat: notification deletion Nov 11, 2025
@Cattn
Copy link
Contributor

Cattn commented Nov 11, 2025

This is a server issue. I am however going to make a change

@Cattn Cattn marked this pull request as ready for review November 11, 2025 02:58
@Cattn Cattn self-requested a review as a code owner November 11, 2025 02:58
Copy link
Contributor

@Cattn Cattn left a comment

Choose a reason for hiding this comment

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

looks good

@Cattn
Copy link
Contributor

Cattn commented Nov 11, 2025

Merging this as it's fine, just waiting on a server fix.

@Cattn Cattn merged commit 6086a77 into main Nov 11, 2025
1 check passed
@miretskyd miretskyd deleted the feat/notif-delete branch November 11, 2025 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants