Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@nikolaas
Copy link
Contributor

Proposed Changes

The PR adds support for the internationalization of the login page.

@nikolaas nikolaas requested a review from konstlepa November 17, 2020 08:04
return err
}

func isOldStyleUserTemplate(root *template.Template) bool {
Copy link
Member

Choose a reason for hiding this comment

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

Mention the issue that causes the creation of this function.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

return errors.Wrapf(err, "failed to parse template %q: %s", name, err)
}

if isOldStyleUserTemplate(root) {
Copy link
Member

Choose a reason for hiding this comment

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

Mention that this code will be deprecated in the future major release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@codecov-io
Copy link

Codecov Report

Merging #16 (428b9bc) into master (b0d037c) will decrease coverage by 0.68%.
The diff coverage is 55.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   73.46%   72.77%   -0.69%     
==========================================
  Files           8        8              
  Lines         539      518      -21     
==========================================
- Hits          396      377      -19     
+ Misses        112      107       -5     
- Partials       31       34       +3     
Impacted Files Coverage Δ
internal/identp/identp.go 77.63% <0.00%> (+0.75%) ⬆️
internal/web/web.go 68.65% <63.33%> (-3.09%) ⬇️
internal/stat/stat.go 69.23% <0.00%> (-4.11%) ⬇️
internal/hydra/hydra.go 55.40% <0.00%> (-2.29%) ⬇️
internal/hydra/login.go 100.00% <0.00%> (ø)
internal/hydra/logout.go 100.00% <0.00%> (ø)
internal/hydra/consent.go 100.00% <0.00%> (ø)
internal/ldapclient/ldapclient.go 72.22% <0.00%> (+1.27%) ⬆️

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 b0d037c...428b9bc. Read the comment docs.

@konstlepa konstlepa merged commit 46ef5bd into i-core:master Mar 5, 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.

3 participants