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

Refactored multi language support & route definition #53

Conversation

AdrianSkierniewski
Copy link
Member

No description provided.

@AdrianSkierniewski AdrianSkierniewski force-pushed the fixed_issues_related_to_language_usage_in_service_provider branch from 9c9dbf1 to 090d504 Compare February 15, 2018 15:41
@coveralls
Copy link

coveralls commented Feb 15, 2018

Coverage Status

Coverage increased (+0.8%) to 81.39% when pulling 403a807 on fixed_issues_related_to_language_usage_in_service_provider into b0576ad on master.

PSkierniewski
PSkierniewski previously approved these changes Feb 15, 2018
Copy link
Member

@PSkierniewski PSkierniewski left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@PSkierniewski PSkierniewski left a comment

Choose a reason for hiding this comment

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

👍

@AdrianSkierniewski AdrianSkierniewski merged commit a307b8a into master Feb 15, 2018
@AdrianSkierniewski AdrianSkierniewski deleted the fixed_issues_related_to_language_usage_in_service_provider branch February 15, 2018 16:28
}

app()->setLocale($language->code);

// We need it to make carbon's diffForHumans work correctly
Carbon::setLocale(app()->getLocale());
Copy link

Choose a reason for hiding this comment

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

I would rewrite it for better simplicity to this

Carbon::setLocale($language->code)

However it is a matter of a taste

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.

None yet

4 participants