Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[email protected]: rating fixed point value #2460

Merged
merged 4 commits into from
Mar 26, 2024

Conversation

mmakwe-onyeka
Copy link
Contributor

Changed
Update the rating fixed point

@github-actions github-actions bot added the molecules This PR changes at least one "molecule" component label Feb 26, 2024
@mmakwe-onyeka mmakwe-onyeka changed the title Update: rating fixed point value [email protected]: rating fixed point value Feb 26, 2024
Charlotte-91
Charlotte-91 previously approved these changes Mar 11, 2024
Copy link
Contributor

@siggerzz siggerzz left a comment

Choose a reason for hiding this comment

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

Looks like you've got a couple unit tests to update before this can be merged. Please reach out if you need a hand with this @mmakwe-onyeka!

@mmakwe-onyeka mmakwe-onyeka force-pushed the f-restaurant-card/update-rating-fixed-point branch from a49c2bc to aef141a Compare March 25, 2024 08:35
@mmakwe-onyeka mmakwe-onyeka requested a review from siggerzz March 25, 2024 09:04
@mmakwe-onyeka
Copy link
Contributor Author

Looks like you've got a couple unit tests to update before this can be merged. Please reach out if you need a hand with this @mmakwe-onyeka!

Thank you @siggerzz ive made the update

@mmakwe-onyeka mmakwe-onyeka changed the title [email protected]: rating fixed point value [email protected]: rating fixed point value Mar 25, 2024
@siggerzz siggerzz merged commit d6c2f5c into master Mar 26, 2024
21 of 22 checks passed
@siggerzz siggerzz deleted the f-restaurant-card/update-rating-fixed-point branch March 26, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
molecules This PR changes at least one "molecule" component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants