Skip to content

Conversation

@jonny22094
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Oct 23, 2018

Codecov Report

Merging #295 into master will decrease coverage by 1.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #295      +/-   ##
==========================================
- Coverage   38.47%   37.21%   -1.27%     
==========================================
  Files          88       99      +11     
  Lines        2773     4109    +1336     
  Branches        6        6              
==========================================
+ Hits         1067     1529     +462     
- Misses       1706     2580     +874

@jonny22094 jonny22094 self-assigned this Oct 23, 2018

## Socket mapping

Mapping is a configuration of hosting which gives us possibility bypass preflighted requests in CORS
Copy link
Member

Choose a reason for hiding this comment

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

Where are the dots at the end of sentences?

Maybe something more like:
"Mapping is a configuration of hosting which gives you a possibility to expose sockets endpoints as a part of the hosting URL. It is also helping to avoid preflighted requests in CORS.

For example:
->
"

## Socket mapping

Mapping is a configuration of hosting which gives us possibility bypass preflighted requests in CORS
Mapping is a configuration of hosting which gives you a possibility to expose sockets endpoints as a part of the hosting URL. It is also helping to avoid preflighted requests in CORS.
Copy link
Member

Choose a reason for hiding this comment

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

Sockets with capital letter.

@jonny22094 jonny22094 merged commit 374c269 into master Oct 26, 2018
@jonny22094 jonny22094 deleted the feat/socket_mapping_docs branch October 26, 2018 08:46
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