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

[DIT-4270] Support pulling multiple formats at once #66

Merged
merged 3 commits into from
Jun 27, 2023

Conversation

azjgard
Copy link
Collaborator

@azjgard azjgard commented Jun 26, 2023

Overview

Adds support for optionally specifying a list of formats in the format property.

image image

Context

https://linear.app/dittowords/issue/DIT-4270/pull-down-strings-and-stringsdict

Test Plan

  • In config.yml, confirm you can pull a single format like format: flat
  • In config.yml, confirm you can pull multiple formats like
format:
  - ios-strings
  - ios-stringsdict

if specified:
  - takes precedence over any value specified for the `format` property
  - fetches and saves data in both `.strings` and `.stringsdict` formats
  - does not generate a `.js` driver file
@azjgard azjgard changed the title [DIT-4270] "iOS Mode": Pull down .strings and .stringsdict files together [DIT-4270] Support generating .strings and .stringsdict files together Jun 26, 2023
@azjgard azjgard changed the title [DIT-4270] Support generating .strings and .stringsdict files together [DIT-4270] Support saving .strings and .stringsdict files together Jun 26, 2023
@azjgard azjgard changed the title [DIT-4270] Support saving .strings and .stringsdict files together [DIT-4270] Support pulling multiple formats at once Jun 26, 2023
@asnewman asnewman self-assigned this Jun 27, 2023
lib/types.ts Outdated
@@ -21,6 +21,7 @@ export type SupportedFormat =
| "icu";

export interface ConfigYAML {
mode?: "ios";
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this for?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oops, left behind from original implementation

lib/types.ts Outdated Show resolved Hide resolved
@azjgard azjgard merged commit 90c15b4 into master Jun 27, 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