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

appx manifest - supported languages #1684

Closed
black-snow opened this issue Jun 16, 2017 · 10 comments · May be fixed by qcif/data-curator#563
Closed

appx manifest - supported languages #1684

black-snow opened this issue Jun 16, 2017 · 10 comments · May be fixed by qcif/data-curator#563

Comments

@black-snow
Copy link
Contributor

It's great to have some improved appx support for I can drop a lot of custom scripts. Now there's one thing I'm still missing and that's support for listing supported languages in the appx manifest. Currently en-US is the default one and that can't be changed easily. (I'm unpacking the appx, changing the manifest, packing, signing.)

See: https://docs.microsoft.com/en-us/windows/uwp/globalizing/manage-language-and-region#specify-the-supported-languages-in-the-apps-manifest

I'd like to state the supported languages in my build config like:

"appx": {
  "publisher": "...",
  "languages": ["en-US", "de-DE", "FR-fr"]
}

Where the first one's the default language. It should default to en-US.

I'll try to file a PR if you aren't faster ;)

@develar
Copy link
Member

develar commented Jun 16, 2017

I'll try to file a PR if you aren't faster ;)

I have assigned you to fix it ;)

@develar
Copy link
Member

develar commented Jun 16, 2017

Our test suite currently doesn't unpack appx and doesn't verify manifest — test will be not easy to write.

@black-snow
Copy link
Contributor Author

Aight.
Shouldn't be too hard if we have makeappx.exe available unless I'm missing something. Haven't looked at the tests yet.

  1. Unpack appx or do this before packing
  2. Regex check (simple) or throw some xml parsing at it (overkill).

@develar
Copy link
Member

develar commented Jun 16, 2017

@black-snow In case of Info.plist we remove some always changed fields and then toMatchSnapshot(). Will be cool if you will add such functionality to test appx.

@black-snow
Copy link
Contributor Author

Thanks for the hint.

@develar
Copy link
Member

develar commented Jun 16, 2017

BTW, so disappointing that we cannot run makeappx on mac/linux :(

@black-snow
Copy link
Contributor Author

This is the first time I'm actually doing something in typescript. Implementation should be done.

After a quick glimpse at jest snapshot testing I'm really confused. Where are those snapshots? How would I add one? How would that snapshot look like? * Serialized* manifest? appx?
Also I'm confused by the appx test. What does it do? It's calling app wich is basically assertPack which doesn't even generate an appx, does it? There's no easy way to hook into single build steps for appx, eh? Guess you were right ...

test will be not easy to write.

@develar
Copy link
Member

develar commented Jun 17, 2017

Jest automatically creates snapshot file on first run. Appx test check that appx produces some files, but appx works only on Windows.

I will grab your changes and will commit :)

develar pushed a commit to develar/electron-builder that referenced this issue Jun 17, 2017
@develar
Copy link
Member

develar commented Jun 17, 2017

@black-snow Thanks!

@black-snow
Copy link
Contributor Author

@develar You're welcome.
I'll be writing my thesis the next couple weeks so, sadly, I won't be looking into appx tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants