Skip to content

Comments

Feature/edit recipe#34

Merged
ascibisz merged 2 commits intomainfrom
feature/edit-recipe
Jul 18, 2025
Merged

Feature/edit recipe#34
ascibisz merged 2 commits intomainfrom
feature/edit-recipe

Conversation

@ascibisz
Copy link
Contributor

@ascibisz ascibisz commented Jul 7, 2025

Problem

We want recipes to be editable in the cellPACK client.
Link to story or ticket

Solution

  • Made the box where the recipe is displayed editable
  • When a user hits submit, we check if the recipe has been changed from its original version. If it hasn't, we proceed as normal. If it has, we enter a new flow for edited recipes
  • A new UUID is generated for the edited recipe, and that recipe is uploaded to the firebase recipes_edited collection.
  • The path to the new, edited recipe in Firebase is sent to the /submit-packing endpoint.
  • Some changes had to be made to cellpack for the server to accept the format of recipes in the recipes_edited collection. Those changes are here

@ascibisz ascibisz force-pushed the feature/edit-recipe branch from cf93a79 to 144b1ee Compare July 7, 2025 23:10
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-07-18 23:48 UTC

@ascibisz ascibisz linked an issue Jul 8, 2025 that may be closed by this pull request
@ascibisz ascibisz marked this pull request as ready for review July 8, 2025 16:36
@ascibisz ascibisz force-pushed the feature/edit-recipe branch from 144b1ee to 04f3111 Compare July 18, 2025 22:47
@ascibisz ascibisz requested a review from rugeli July 18, 2025 22:49
@ascibisz ascibisz merged commit 6dfbb40 into main Jul 18, 2025
1 check passed
@meganrm meganrm deleted the feature/edit-recipe branch November 13, 2025 19:31
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.

Upload recipes from client to firebase

2 participants