Skip to content

Conversation

@carolinacmoravia
Copy link
Contributor

Hello, @ardalis,

Translator has reported possible source content issue.

Please, help to check my comment tag added into the article and reply with explanation if source text should be fixed, or it is by design. If fix is needed, please, let me know either if you would like me to fix it in this PR, or if you will fix it in another PR. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR.

Many thanks in advance.

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

Hello, @ardalis, 

Translator has reported possible source content issue.  

Please, help to check my comment tag added into the article and reply with explanation if source text should be fixed, or it is by design. If fix is needed, please, let me know either if you would like me to fix it in this PR, or if you will fix it in another PR. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR. 

Many thanks in advance.
@mairaw mairaw closed this Apr 9, 2018
@mairaw mairaw reopened this Apr 9, 2018
@mairaw
Copy link
Contributor

mairaw commented Apr 9, 2018

@ardalis @CESARDELATORRE @rachelappel can you guys with this fix? Not sure what we'd like to say here.
Should we say "that makes it simple to add real-time"?

The sentence does read a bit weird:
"ASP.NET Core SignalR is a library that makes it simple to kind of real-time server-to-client communication functionality to your applications."

@rachelappel
Copy link

@mairaw "ASP.NET Core SignalR is a library that makes it simple to add real-time server-to-client communication functionality to your applications."

Copy link

@rachelappel rachelappel left a comment

Choose a reason for hiding this comment

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

"ASP.NET Core SignalR is a library that makes it simple to add real-time server-to-client communication functionality to your applications."

@ardalis
Copy link
Contributor

ardalis commented Apr 9, 2018

Rachel's update is good. Not sure where "kind of" came from...

@mairaw
Copy link
Contributor

mairaw commented Apr 10, 2018

@carolinacmoravia do you want to handle the fix then with the suggested change?

Replace "to kind of" with "to add". Thanks for flagging this!

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

Please remove the Loc comment and replace "to kind of" with "to add". Thanks!

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

I've made the changes on your behalf @carolinacmoravia.

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.

4 participants