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

fix: Stars icon has larger clickable area #145

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

Resulte
Copy link

@Resulte Resulte commented Apr 24, 2022

Fixes : #26

Description :

Stars icon has larger clickable area

Screenshot of changes :

Screen Shot 2022-04-24 at 20 30 54

Please ensure Checkmark the following :

  • - Tested Localy
  • - Checked in IE11 and Firefox
  • - Checked in mobile
  • - Backend work
  • - Frontend work
  • - Bug Fixes
  • - Enhancements

guxiaoyang_s20_307 and others added 18 commits November 19, 2020 15:26
Signed-off-by: guxiaoyang_s20_307 <[email protected]>
Issue:
    1. Api "https://api.github.com/user/emails" 不再支持 query params 方式追加 access_token参数;
    2. Api "https://api.github.com/user" 不再支持 query params 方式追加 access_token参数;

Resolve:
    以上两接口均按 [github 文档-user](https://docs.github.com/en/free-pro-team@latest/rest/reference/users#list-email-addresses-for-the-authenticated-user)改为 在 header 中 添加
      - ("Authorization", "token " + access_token);
      - ("access", "application/vnd.github.v3+json");
    得以解决。
* fixed some typos and pointer

* fixed some typos and pointer
Two apis of github oauth changed which cause login failed
Add more details about setting up OAuth application
Aligning the submit button in the add policy page - (Solves - casbin#125)
Signed-off-by: Pranav Kulshrestha <[email protected]>
Center aligning the submit button for the add and edit page - (Solves - casbin#143)
@casbin-bot
Copy link
Member

@tangyang9464 @closetool @sagilio please review

@CLAassistant
Copy link

CLAassistant commented Apr 24, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 6 committers have signed the CLA.

✅ Resulte
✅ hsluoyz
❌ guxiaoyang_s20_307
❌ DeadSpoon18
❌ pranav1698
❌ tufatao


guxiaoyang_s20_307 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Resulte
Copy link
Author

Resulte commented Apr 25, 2022

@hsluoyz please review.

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.

Stars icon has larger clickable area
7 participants