Skip to content

Update nl.yml for v4#8195

Merged
javierjulio merged 2 commits into
activeadmin:masterfrom
caifara:v4_nl
Dec 30, 2023
Merged

Update nl.yml for v4#8195
javierjulio merged 2 commits into
activeadmin:masterfrom
caifara:v4_nl

Conversation

@caifara

@caifara caifara commented Dec 30, 2023

Copy link
Copy Markdown
Contributor

Adapts V4 i18n changes for Dutch translations.

Also: chose "reactie" instead of "commentaar" (as both translations were in use).

V4 introduces changes in pagination.

Also: chose "reactie" instead of "commentaar" (both translations were in use).
@codecov

codecov Bot commented Dec 30, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8fd0a19) 99.03% compared to head (ebf3edf) 99.03%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #8195   +/-   ##
=======================================
  Coverage   99.03%   99.03%           
=======================================
  Files         151      151           
  Lines        4150     4150           
=======================================
  Hits         4110     4110           
  Misses         40       40           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@javierjulio javierjulio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@caifara thank you. 👏🏻 This is missing a single v4 default change. Would you mind translating the status_tag.unset from "No" to "Unknown" in the Dutch equivalent? The default was changed to separate false from nil, e.g. in the case of booleans.

@caifara

caifara commented Dec 30, 2023

Copy link
Copy Markdown
Contributor Author

@javierjulio done, thank you for all your effort!

@javierjulio javierjulio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@caifara thank you for your support and contribution! I really appreciate it. ❤️

@javierjulio javierjulio merged commit f20bf0c into activeadmin:master Dec 30, 2023
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.

2 participants