-
Notifications
You must be signed in to change notification settings - Fork 143
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
Editorial changes to Safe Method With Body #2859
base: main
Are you sure you want to change the base?
Conversation
I hope that I have followed the proper procedures for proposing (and labeling) an editorial suggestion. I hope that this is helpful! |
Yes, that's the right procedure! I think the change goes into the right direction. Two issues though:
|
A minor typographical fix and slight rephrasing of limitations of current query-embedded-in-request-URI method.
My goal was to reflow the content into those three bullets. Did I accidentally delete other content while doing that? If so, I am very sorry!
I believe that I fixed this issue! Sorry! I made sure that everything compiled with this version (as opposed to the first version -- sorry!) I hope that this helps! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some more suggestions, but small ones.
Accept @martinthomson helpful feedback! Co-authored-by: Martin Thomson <[email protected]>
Accepting additional feedback - thanks. Co-authored-by: Martin Thomson <[email protected]>
Thank you @martinthomson ! I incorporated all those helpful suggestions! |
Will,
|
Absolutely! I was just trying to respond to @martinthomson !! I will send another version in the next few hours! Sorry for the trouble. |
A minor typographical fix and slight rephrasing of limitations of current query-embedded-in-request-URI method.