Skip to content

Conversation

@jjoonleo
Copy link
Contributor

@jjoonleo jjoonleo commented Dec 31, 2025

Describe your changes

change HTTP method from POST to PUT for updating preparation schedules
Implement PreparationEditDraftCubit for managing draft state in preparation editing flow

Issue ticket number and link

#327

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.

Screenshots (if appropriate):


Note

Prep schedule updates and edit flow

  • Switch updatePreparationByScheduleId API call from POST to PUT.
  • Introduce PreparationEditDraftCubit to hold draft state across /preparationEdit navigation; editor saves to draft and caller reads/clears after return.
  • Update PreparationEditForm to read/set draft, and router /preparationEdit to no longer require extra param.
  • Replace Dismissible list with SwipeActionCell swipe-to-delete in PreparationFormReorderableList; remove enableDismissible flag and delete preparation_form_reorderable_list_dissmissible.dart.
  • Update schedule form to open editor using draft, apply changes on return, and clear stale drafts.

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

@jjoonleo jjoonleo linked an issue Dec 31, 2025 that may be closed by this pull request
4 tasks
@jjoonleo jjoonleo removed the request for review from SeoHyeonSim December 31, 2025 08:37
@github-actions
Copy link

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

https://ontime-c63f1--pr336-333-test-preparation-tzdnddv6.web.app

(expires Wed, 07 Jan 2026 08:40:43 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7d60a2eae4c4a5839220ed6a57552a3b61cce5a3

@github-actions
Copy link

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

https://on-time-front-widgetbook--pr336-333-test-preparation-b6p5j8ku.web.app

(expires Wed, 07 Jan 2026 08:41:11 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e53128a376fed0209d449279de9c0d94c83cdd8b

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.

[Test]: PreparationFormCreateList Component

2 participants