Skip to content

fix: detail persistence and ingredient render#138

Merged
alecorvec merged 2 commits intodevfrom
135-some-ingredients-dont-show-up
Apr 11, 2025
Merged

fix: detail persistence and ingredient render#138
alecorvec merged 2 commits intodevfrom
135-some-ingredients-dont-show-up

Conversation

@alecorvec
Copy link
Copy Markdown
Contributor

Describe your changes

Fixed the various bugs in the details and ingredients pages

How to test the feature

The pages should be back to previous states, functionning properly.

Issue ticket number and link

closes #135

Checklist before requesting a review

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have performed a self-review of my code
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Documentation has been updated as required
    If a single item in this checklist is not checked, the pull request cannot be accepted

@alecorvec alecorvec requested a review from JulesGresset April 10, 2025 19:09
@alecorvec alecorvec self-assigned this Apr 10, 2025
@alecorvec alecorvec linked an issue Apr 10, 2025 that may be closed by this pull request
@alecorvec alecorvec enabled auto-merge April 11, 2025 14:22
@alecorvec alecorvec merged commit 817fe35 into dev Apr 11, 2025
7 checks passed
@alecorvec alecorvec deleted the 135-some-ingredients-dont-show-up branch April 11, 2025 16:55
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.

Some ingredients don't show up

2 participants