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

url /get_cookie, not /get_cookies #395

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

shouldnotappearcalm
Copy link
Contributor

@shouldnotappearcalm shouldnotappearcalm commented Apr 6, 2024

🤔 What is the nature of this change? / 这个变动的性质是?

  • New feature / 新特性提交
  • Fix bug / bug 修复
  • Refactor code or style / 重构代码或样式
  • Performance optimization / 性能优化
  • Build optimization / 构建优化
  • Website, documentation, demo improvements / 网站、文档、Demo 改进
  • Test related / 测试相关
  • Other / 其他

🔗 Related Issue / 相关 Issue

暂无

💡 Background or solution / 需求背景和解决方案

代码中是 /get_cookie,但是文档中写的url是 get_cookies,更正了README中的写法

@app.post("/get_cookie")

@shouldnotappearcalm
Copy link
Contributor Author

PR Review

⏱️ Estimated effort to review [1-5]

1, because the PR involves a simple change in documentation text, correcting an API endpoint in two markdown files. The change is straightforward and does not involve complex code modifications.

🧪 Relevant tests

No

🔍 Possible issues

No

🔒 Security concerns

No

Code feedback:

✨ Review tool usage guide:

Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.

The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

  • When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...
[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.

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.

1 participant