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

#4265 Replace echo.websocket.org -> ws.ifelse.io #4266

Merged
merged 1 commit into from
May 8, 2023

Conversation

SpyrosKou
Copy link
Contributor

  • Replaced usages of ws://echo.websocket.org echo websocket that is no longer available, with ws://ws.ifelse.io that is currently available.

References #4265

@lightbend-cla-validator
Copy link

Hi @SpyrosKou,

Thank you for your contribution! We really value the time you've taken to put this together.

Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:

https://www.lightbend.com/contribute/cla

@SpyrosKou
Copy link
Contributor Author

Hi @SpyrosKou,

Thank you for your contribution! We really value the time you've taken to put this together.

Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:

https://www.lightbend.com/contribute/cla

The Lightbend CLA is signed.

@SpyrosKou SpyrosKou force-pushed the ReplaceWebsocketServerInDocs branch from 8916693 to 6ffcec8 Compare May 8, 2023 12:01
@SpyrosKou SpyrosKou force-pushed the ReplaceWebsocketServerInDocs branch from 6ffcec8 to 7484b5a Compare May 8, 2023 12:01
Copy link
Member

@johanandren johanandren left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@johanandren johanandren merged commit 135d644 into akka:main May 8, 2023
@SpyrosKou SpyrosKou deleted the ReplaceWebsocketServerInDocs branch May 8, 2023 12:50
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