tradingview / charting-library-examples

Examples of Charting Library integrations with other libraries, frameworks and data transports
MIT License
1.33k stars 744 forks source link

[iOS] Font not supported by platform #216

Open Bardiamist opened 3 years ago

Bardiamist commented 3 years ago

Checklist

Describe the bug

My team tried to deploy app to TestFlight and we got next

Dear Developer,
We identified one or more issues with a recent submission for TestFlight review for your app, "APP_NAME" 1.0.0 (1). Please correct the following issues, then upload again.
ITMS-90853: Invalid font - There was an error validating the font at 'my.app/tradingView/charting_library/bundles/40f9a03d174178efb12303caa9bc7cd8.woff2'. Font not supported by platform.
ITMS-90853: Invalid font - There was an error validating the font at 'my.app/tradingView/charting_library/bundles/3293616ec0c605c7c2db25829a0a509e.woff'. Font not supported by platform.
Best regards,
The App Store Team

To Reproduce

Steps to reproduce the behavior:

  1. Try to upload example to TestFlight

Expected behavior

App should be uploaded successfuly.

timocov commented 3 years ago

It seems that iOS supports that type of fonts since 10 version https://caniuse.com/?search=woff2. It's possible that the TestFlight team interprets that font like you're trying to use them inside the native app instead inside a webview?

Bardiamist commented 3 years ago

It's possible that the TestFlight team interprets that font like you're trying to use them inside the native app instead inside a webview?

Should be yes. But other developers started to have the same problem from Xcode 12. I see that the same error code can happen also when the TTF font is invalid. So maybe need to replace WOFF files by valid (according to Apple).

timocov commented 3 years ago

So maybe need to replace WOFF files by valid (according to Apple).

Can you please elaborate what is "valid according by Apple"? I mean, maybe we can fix it somehow from our end (nevertheless I believe these fonts are valid though).

timocov commented 3 years ago

@Bardiamist can you please check suggestion from https://stackoverflow.com/questions/64348482/itms-90853-invalid-font-on-testflight-external-testing?

@makedonsky94 can you please take a look?

Bardiamist commented 3 years ago

valid according by Apple

I don't know why Apple thinks this files is not supported.

@Bardiamist can you please check suggestion from https://stackoverflow.com/questions/64348482/itms-90853-invalid-font-on-testflight-external-testing?

I tried to use this CR8 Software program. But this program don't support WOFF.

Bardiamist commented 3 years ago

Now I just removed both files. Where is using that fonts?

timocov commented 3 years ago

I believe it's different format of the same font and it's used in icon tool:

image