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

Multi-language documentation for Forwarding Messages with @SendTo example #827

Merged
merged 12 commits into from
Aug 28, 2023

Conversation

wetted
Copy link
Contributor

@wetted wetted commented Aug 18, 2023

No description provided.

@wetted wetted self-assigned this Aug 18, 2023
@wetted
Copy link
Contributor Author

wetted commented Aug 18, 2023

This could probably use some tests, although these docs examples never had any to begin with.

Copy link
Contributor

@sdelamo sdelamo left a comment

Choose a reason for hiding this comment

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

Please, add tests

@wetted wetted marked this pull request as draft August 21, 2023 22:19
Copy link
Contributor

@sdelamo sdelamo left a comment

Choose a reason for hiding this comment

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

is this PR ready for review?

@wetted
Copy link
Contributor Author

wetted commented Aug 22, 2023

is this PR ready for review?

no, still meeds tests

@wetted wetted marked this pull request as ready for review August 25, 2023 17:40
@sonarcloud
Copy link

sonarcloud bot commented Aug 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sdelamo sdelamo merged commit 7141fdc into master Aug 28, 2023
7 checks passed
@sdelamo sdelamo deleted the docs-sendto branch August 28, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants