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(web): fix the mismatch in return types of EchoService.postEvent api stub and the actual api from echo #1885

Merged

Conversation

kirangodishala
Copy link
Contributor

  • added a test to demonstrate the issue
  • fixed the issue by correcting the return type of EchoService.postEvent api stub

…EchoService.postEvent api stub and the actual api from echo.
@kirangodishala kirangodishala requested a review from dbyron-sf March 3, 2025 19:52
@dbyron-sf dbyron-sf added the ready to merge Approved and ready for merge label Mar 3, 2025
@mergify mergify bot added the auto merged label Mar 3, 2025
@mergify mergify bot merged commit ed77027 into spinnaker:master Mar 3, 2025
5 checks passed
@dbyron-sf
Copy link
Contributor

@Mergifyio backport release-1.37.x

Copy link
Contributor

mergify bot commented Mar 3, 2025

backport release-1.37.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Mar 3, 2025
…pi stub and the actual api from echo (#1885)

* test(web): add a test to demonstrate the mismatch in return types of EchoService.postEvent api stub and the actual api from echo.

* fix(web): fix the mismatch in return types of EchoService.postEvent api stub and the actual api from echo.

(cherry picked from commit ed77027)
mergify bot added a commit that referenced this pull request Mar 3, 2025
…pi stub and the actual api from echo (#1885) (#1886)

* test(web): add a test to demonstrate the mismatch in return types of EchoService.postEvent api stub and the actual api from echo.

* fix(web): fix the mismatch in return types of EchoService.postEvent api stub and the actual api from echo.

(cherry picked from commit ed77027)

Co-authored-by: Kiran Godishala <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants