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

Fix duplicated response #111

Merged
merged 2 commits into from
Jul 9, 2024
Merged

Fix duplicated response #111

merged 2 commits into from
Jul 9, 2024

Conversation

llagerlof
Copy link
Contributor

This PR fix the issue #110

@mefengl
Copy link

mefengl commented Jul 4, 2024

It seems you accidentally included another commit from a different pull request in this pull request :)

@llagerlof
Copy link
Contributor Author

Owner of this repository, about this PR, merge only the commit "Fix duplicated response" if you can.

@steve8708
Copy link
Contributor

@llagerlof I don't know of an option to merge just the commit, I'll need you to remoce that commit (e.g. through a rebase)

e.g.

git reset --hard HEAD~1
git push origin --force-with-lease

@steve8708
Copy link
Contributor

also, thanks for the fix!

@llagerlof
Copy link
Contributor Author

@steve8708 I will write more to fix this commit than the code itself. You can close this PR and make the fix (it's just one line), no problem for me.

Thanks!

@steve8708
Copy link
Contributor

just pushed the fix, thanks @llagerlof !

@steve8708 steve8708 merged commit f5787e3 into BuilderIO:main Jul 9, 2024
2 checks passed
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.

3 participants