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

275817 - Use i18n islet resource #79

Merged
merged 11 commits into from
Dec 13, 2023

Conversation

MBausson
Copy link
Contributor

@MBausson MBausson commented Oct 6, 2023

No description provided.

@nicolas-brousse nicolas-brousse self-requested a review October 9, 2023 12:12
Copy link
Collaborator

@nicolas-brousse nicolas-brousse left a comment

Choose a reason for hiding this comment

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

@MBausson I've to check a bit more, but here is my first review

app/views/islets/_form.html.erb Outdated Show resolved Hide resolved
Copy link
Collaborator

@nicolas-brousse nicolas-brousse left a comment

Choose a reason for hiding this comment

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

@MBausson some cleans to do. I've to think a bit more for export_button to be sure of the direction we'll took

config/locales/activerecord.fr.yml Outdated Show resolved Hide resolved
config/locales/activerecord.fr.yml Show resolved Hide resolved
@MBausson MBausson force-pushed the 275817-i18n-islet-resource branch from be4a98d to a2a2076 Compare October 31, 2023 10:05
@MBausson MBausson force-pushed the 275817-i18n-islet-resource branch 2 times, most recently from d43a894 to c806911 Compare November 10, 2023 13:35
Copy link
Collaborator

@B-Rass B-Rass left a comment

Choose a reason for hiding this comment

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

Good!

@MBausson MBausson force-pushed the 275817-i18n-islet-resource branch from c806911 to 864024b Compare December 6, 2023 09:26
app/views/islets/index.html.erb Show resolved Hide resolved
app/views/islets/index.html.erb Outdated Show resolved Hide resolved
@MBausson MBausson force-pushed the 275817-i18n-islet-resource branch from eff974d to 3415823 Compare December 6, 2023 11:38
Copy link
Collaborator

@nicolas-brousse nicolas-brousse left a comment

Choose a reason for hiding this comment

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

@MBausson one last change otherwise LGTM

app/views/islets/index.html.erb Outdated Show resolved Hide resolved
config/locales/activerecord.fr.yml Outdated Show resolved Hide resolved
@nicolas-brousse nicolas-brousse force-pushed the 275817-i18n-islet-resource branch from d50bfd3 to 1792dec Compare December 13, 2023 14:56
@nicolas-brousse nicolas-brousse merged commit 80dde74 into nanego:master Dec 13, 2023
1 check passed
@nicolas-brousse nicolas-brousse deleted the 275817-i18n-islet-resource branch December 13, 2023 15:42
nicolas-brousse added a commit to pantographe/my-dcim that referenced this pull request Dec 20, 2023
Co-authored-by: Maximilien Bausson <[email protected]>
Co-authored-by: Nicolas Brousse <[email protected]>
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