Skip to content
This repository has been archived by the owner on Apr 11, 2023. It is now read-only.

fix: Update signupText and signinText to logos #142

Merged
merged 1 commit into from
Dec 21, 2021

Conversation

talwinder50
Copy link
Member

@talwinder50 talwinder50 commented Dec 21, 2021

Change tested in local wallet
Screen Shot 2021-12-21 at 1 13 17 PM

Screen Shot 2021-12-21 at 1 13 09 PM

closes #141

Signed-off-by: talwinder50 talwinderkaur50@gmail.com

@cla-bot cla-bot bot added the cla-signed label Dec 21, 2021
@codecov
Copy link

codecov bot commented Dec 21, 2021

Codecov Report

Merging #142 (2b268bc) into main (39b4341) will not change coverage.
The diff coverage is 22.22%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #142   +/-   ##
=======================================
  Coverage   89.25%   89.25%           
=======================================
  Files           9        9           
  Lines         968      968           
=======================================
  Hits          864      864           
  Misses         76       76           
  Partials       28       28           
Impacted Files Coverage Δ
cmd/auth-rest/startcmd/start.go 86.60% <0.00%> (ø)
pkg/restapi/operation/operations.go 94.77% <100.00%> (ø)

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 39b4341...2b268bc. Read the comment docs.

LogoURL string `yaml:"logoURL"`
SignUpLogo string `yaml:"signUpLogo"`
SignInLogo string `yaml:"signInLogo"`
Order string `yaml:"order"`
Copy link
Member

Choose a reason for hiding this comment

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

Should this be an int ?

SignInText string `yaml:"signInText"`
LogoURL string `yaml:"logoURL"`
SignUpLogo string `yaml:"signUpLogo"`
SignInLogo string `yaml:"signInLogo"`
Copy link
Member

Choose a reason for hiding this comment

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

Can we add prefix URL ? since this is pointing to an url

closes trustbloc#141

Signed-off-by: talwinder50 <talwinderkaur50@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update oidc config
3 participants