Skip to content
This repository has been archived by the owner on Feb 5, 2021. It is now read-only.

This pull request refers to a #6 issue: 'add i18n to the whole application' #87

Merged
merged 8 commits into from
May 15, 2018

Conversation

danielgoncalvesti
Copy link
Contributor

I made all changes that you suggested to me. I've already tested in the Safari, Firefox and Chrome browsers.

@danielgoncalvesti danielgoncalvesti changed the title this commit refers to a #6 issue: 'add i18n to the whole application'" This pull request refers to a #6 issue: 'add i18n to the whole application' May 9, 2018
@codecov-io
Copy link

Codecov Report

Merging #87 into master will decrease coverage by 0.48%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
- Coverage   41.42%   40.93%   -0.49%     
==========================================
  Files          22       22              
  Lines         338      342       +4     
  Branches        5        3       -2     
==========================================
  Hits          140      140              
- Misses        198      202       +4
Impacted Files Coverage Δ
app/controllers/admin/CategoryAdmin.scala 33.33% <ø> (ø) ⬆️
app/controllers/Application.scala 43.75% <0%> (-6.25%) ⬇️
app/controllers/admin/ItemAdmin.scala 34.61% <0%> (ø) ⬆️

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 e6c70b1...00d6641. Read the comment docs.

@jcranky
Copy link
Owner

jcranky commented May 15, 2018

thanks and sorry for the delay :) 👍

@jcranky jcranky merged commit a566330 into jcranky:master May 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants