Skip to content

fix[UI]: improve consent page#907

Merged
XVincentX merged 5 commits intoExpressGateway:masterfrom
ravikp7:oauth-consent-page
Apr 28, 2019
Merged

fix[UI]: improve consent page#907
XVincentX merged 5 commits intoExpressGateway:masterfrom
ravikp7:oauth-consent-page

Conversation

@ravikp7
Copy link
Copy Markdown
Contributor

@ravikp7 ravikp7 commented Apr 17, 2019

fixes #739

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 2019

Codecov Report

Merging #907 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #907      +/-   ##
==========================================
+ Coverage   88.85%   88.85%   +<.01%     
==========================================
  Files         137      137              
  Lines        3722     3724       +2     
==========================================
+ Hits         3307     3309       +2     
  Misses        415      415
Impacted Files Coverage Δ
lib/policies/oauth2/oauth2-routes.js 92.59% <100%> (+0.59%) ⬆️
lib/policies/oauth2/oauth2-server.js 93.75% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9cf407f...aea54b3. Read the comment docs.

@XVincentX
Copy link
Copy Markdown
Member

Hey @ravikp7 — good start. There are couple of additional things I'd like to see here though

  1. The scope consent page is not styled. It'd be great to style this as well so that the flow is consistent
  2. It'd be great to be consistent with our website styling, in particular the font style and the colours. Also, I wouldn't mind having a logo somewhere up!

I know this can be annoying — just let me know in case you're not up for it and I will take it over the finish line.

@ravikp7
Copy link
Copy Markdown
Contributor Author

ravikp7 commented Apr 17, 2019

@XVincentX yeah, the scope consent page is still pending, will take care of that as well as the styling and logo

@ravikp7 ravikp7 force-pushed the oauth-consent-page branch from 055e7d3 to c474640 Compare April 17, 2019 19:24
@ravikp7
Copy link
Copy Markdown
Contributor Author

ravikp7 commented Apr 17, 2019

  • Logo, consistent font and color

Screenshot from 2019-04-18 00-53-23

- add styling and logo
- show requesting scopes in consent page

fixes ExpressGateway#739
@ravikp7 ravikp7 force-pushed the oauth-consent-page branch from c474640 to 2681d74 Compare April 22, 2019 13:46
@ravikp7
Copy link
Copy Markdown
Contributor Author

ravikp7 commented Apr 22, 2019

  • Show requesting scopes

Screenshot from 2019-04-22 19-11-35

@ravikp7 ravikp7 marked this pull request as ready for review April 22, 2019 13:49
@XVincentX
Copy link
Copy Markdown
Member

Sweet! I'll review and merge!

Copy link
Copy Markdown
Member

@XVincentX XVincentX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've pushed some minor css fixes but other than that good job @ravikp7.

Thanks a lot for this!

@XVincentX XVincentX merged commit 268bdf7 into ExpressGateway:master Apr 28, 2019
gatherchou pushed a commit to yilu-tech/express-gateway that referenced this pull request Jul 29, 2021
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.

Consent page improvements: minimal style + scope show

2 participants