Skip to content

Fixes #36335 - Migrate Transifex Config for v3 API#10534

Merged
wbclark merged 1 commit intoKatello:masterfrom
wbclark:36335_migrate_transifex
May 2, 2023
Merged

Fixes #36335 - Migrate Transifex Config for v3 API#10534
wbclark merged 1 commit intoKatello:masterfrom
wbclark:36335_migrate_transifex

Conversation

@wbclark
Copy link
Contributor

@wbclark wbclark commented Apr 26, 2023

Considerations taken when implementing this change?

  1. Created transifex account and joined foreman organization
  2. Created API token at https://app.transifex.com/user/settings/api/
  3. Installed https://github.com/transifex/cli and copied binary to /usr/local/bin/
  4. ran tx migrate and entered API token when prompted

What are the testing steps for this pull request?

  1. You need a transifex account and API token as described above
  2. You need to install https://github.com/transifex/cli as described above
  3. You can test the new configuration with tx pull --minimum-perc 50 --all or similar

@theforeman-bot
Copy link

Issues: #36335

.tx/config Outdated
@@ -1,14 +1,14 @@
[main]
host = https://www.transifex.com
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is still wrong, it turns out....

...but I fixed it with transifex/cli#187

I'll re-run the migration with the fix so that this gets properly updated to host = https://app.transifex.com

@wbclark wbclark force-pushed the 36335_migrate_transifex branch from 614ce0d to c092ff0 Compare April 27, 2023 11:44
Copy link
Member

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

tx-pull worked for me!

@wbclark wbclark merged commit cdc68b0 into Katello:master May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants