Skip to content
This repository has been archived by the owner on Oct 12, 2024. It is now read-only.

Fix textdomain #136

Closed
wants to merge 1 commit into from
Closed

Conversation

bastienho
Copy link

@bastienho bastienho commented Dec 1, 2021

Textdomain was wrong.

According to WordPress translation standars, textdomain have to be the same as plugin slug.

So woocommerce-local-pickup-time should be replaced by woocommerce-local-pickup-time-select

All Submissions:

Changes proposed in this Pull Request:

Closes #105 .

How to test the changes in this Pull Request:

  1. Set site locale to fr_FR
  2. Go to Dashboard > Upgrades
  3. Update translations

@timnolte
Copy link
Member

This fixes items which are not an issue, and breaks things in some files.

@timnolte timnolte closed this Dec 24, 2021
@bastienho
Copy link
Author

Can you explain which files are broken?

Without this fix, the plugin will never be translated the right way.

@timnolte
Copy link
Member

@bastienho you made blanket find/replace changes to files that have nothing to do with the text domain.

As I've checked you are correct that there appears to have been a long standing misunderstanding with the requirements for the text domain which are clear that it must match the plugin slug.

So while this does in fact need to be fixed it can't be blindly done using a global find/replace. As you should see clearly the GitHub repository isn't the same as the plugin slug. Changing things in README files and the GitHub Actions for CI/CD should not be done.

@bastienho
Copy link
Author

My bad.

I've checked the plugin locally and didn't see any broks, but in fact, I haven't teststed CI/CD stuff.

I'll take a look for a more accurate replacement.

@timnolte
Copy link
Member

I'm working on some development environment and a build updates. It's a simple matter of changing the main text domain specified and running the npm run check:i18n and the places that need an update will be reported. I've made the appropriate changes in my pending changes.

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.

French translation po file availlable
2 participants