Skip to content

Conversation

@markdembo
Copy link
Contributor

Copy link
Contributor

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @markdembo, I've left some comments as a few things seem a bit off to me 🙂

@dario-piotrowicz
Copy link
Contributor

PS: also the formatting seems wrong, you can fix it by running npm run prettier:fix 🙂

markdembo and others added 2 commits March 6, 2025 12:12
Co-authored-by: Dario Piotrowicz <dario.piotrowicz@gmail.com>
@markdembo
Copy link
Contributor Author

Much appreciate the feedback @dario-piotrowicz - applied all requested changes.

Copy link
Contributor

@dario-piotrowicz dario-piotrowicz left a comment

Choose a reason for hiding this comment

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

Thanks a lot @markdembo for the changes 🫶

Sorry there is one last small missing thing, besides that this is good to go for me 😄

Co-authored-by: Dario Piotrowicz <dario.piotrowicz@gmail.com>
@dario-piotrowicz dario-piotrowicz merged commit 33229c5 into opennextjs:main Mar 6, 2025
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.

2 participants