-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
feat: implement set_cookie
function
#870
Conversation
@VishnuSanal is attempting to deploy a commit to the sparckles Team on Vercel. A member of the Team first needs to authorize it. |
CodSpeed Performance ReportMerging #870 will not alter performanceComparing Summary
Benchmarks breakdown
|
@VishnuSanal , could you add some integration tests? |
@VishnuSanal , great work! Could you please update the Also, could you add relevant docs to the documentation? |
robyn/robyn.pyi
Outdated
@@ -285,6 +285,8 @@ class Response: | |||
headers (Union[Headers, dict]): The headers of the response or Headers directly. e.g. {"Content-Type": "application/json"} | |||
description (Union[str, bytes]): The body of the response. If the response is a JSON, it will be a dict. | |||
file_path (Optional[str]): The file path of the response. e.g. /home/user/file.txt | |||
|
|||
set_cookie (Callable): The function to set a cookie (onto headers). |
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.
This is not needed. We only document the attributes
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.
LGTM 👍
Description
This PR fixes #854