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

locale based pagination strings #74

Closed
wants to merge 2 commits into from
Closed

Conversation

NathanPJF
Copy link
Contributor

Replace hardcoded strings with pagination translations.

@NathanPJF
Copy link
Contributor Author

hmmm... Cirlce CI says:

Unused keys:
        general.pagination.previous
       	general.pagination.next
     # ./spec/i18n_parity_spec.rb:28:in `block (2 levels) in <top (required)>'

I'm using them... but assigning them to variables first because I can't write:

{{ paginate | default_pagination: previous: 'general.pagination.previous' | t }}

@NathanPJF NathanPJF mentioned this pull request Jan 5, 2017
@NathanPJF
Copy link
Contributor Author

Ah, has to do with how the test define FILE_KEYS. The match_keys function is only counting uses of {{ * | t}} : https://github.com/Shopify/slate/blob/master/spec/helpers/i18n_helper.rb#L4

So I'm going to close this in favour of the other PR.

Feel it's not worth writing a test that looks for cases of assign var = <some translation> and then making sure that var is used.

@NathanPJF NathanPJF closed this Jan 6, 2017
@NathanPJF NathanPJF deleted the pagination-locale branch January 6, 2017 00:01
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* add theme kit version to version command

* update slate to output the same format as theme kit version

* change framework to toolkit
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* add theme kit version to version command

* update slate to output the same format as theme kit version

* change framework to toolkit
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
* add theme kit version to version command

* update slate to output the same format as theme kit version

* change framework to toolkit
@lock
Copy link

lock bot commented Oct 26, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 26, 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.

1 participant