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.

Sign up for GitHub

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

Jump to bottom

docs: improve jwt doc #369

Merged
merged 1 commit into from
Oct 14, 2022
Merged

docs: improve jwt doc #369

merged 1 commit into from
Oct 14, 2022

Conversation

BaiZe1998
Copy link
Contributor

@BaiZe1998 BaiZe1998 commented Sep 27, 2022 •
edited

What type of PR is this?

doc

What this PR does / why we need it (en: English/zh: Chinese):

en: improve jwt doc for Hertz
zh: 完善 Hertz 的 jwt 扩展的中文文档

@li-jin-gou
Copy link
Member

一个结构可以参考下

- 安装

- 简单完整的代码示例

- 配置
    - 常用字段进行介绍的表格 
    - 针对每个配置的用途进行介绍

- 完整的Example

调整一下结构，然后补充一下 Example,另外配置的详细描述可以对齐一下

@li-jin-gou li-jin-gou changed the title [WIP]doc: improve jwt doc [WIP]docs: improve jwt doc Sep 28, 2022
@BaiZe1998 BaiZe1998 force-pushed the docs/jwt_improve branch 2 times, most recently from 52ae7d1 to 42dff65 Compare October 1, 2022 07:19
@li-jin-gou
Copy link
Member

+英文

@li-jin-gou li-jin-gou changed the title [WIP]docs: improve jwt doc docs: improve jwt doc Oct 11, 2022
@li-jin-gou li-jin-gou mentioned this pull request Oct 11, 2022
Merged
This was referenced Oct 11, 2022
Merged
Merged
@li-jin-gou li-jin-gou mentioned this pull request Oct 11, 2022
Merged
@li-jin-gou li-jin-gou mentioned this pull request Oct 12, 2022
Merged
@li-jin-gou
Copy link
Member

LGTM

@li-jin-gou li-jin-gou merged commit f1f58a9 into cloudwego:main Oct 14, 2022
@L2ncE L2ncE mentioned this pull request Oct 15, 2022
Closed
3 tasks
@BaiZe1998 BaiZe1998 mentioned this pull request Oct 18, 2022
Closed
3 tasks
@justlorain justlorain mentioned this pull request Nov 12, 2022
Closed
3 tasks
@L2ncE L2ncE mentioned this pull request Dec 23, 2022
Closed
@ViolaPioggia ViolaPioggia mentioned this pull request Apr 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Skyenought Skyenought Skyenought left review comments

@justlorain justlorain justlorain left review comments

@L2ncE L2ncE L2ncE left review comments

@welkeyever welkeyever welkeyever approved these changes

@li-jin-gou li-jin-gou li-jin-gou approved these changes

Assignees
No one assigned
Labels
Hertz
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@BaiZe1998 @li-jin-gou @welkeyever @Skyenought @justlorain @L2ncE @GuangmingLuo