Sitelet https://web.archive.org/web/20211125171030/https://github.com/NativeScript/NativeScript/pull/9543
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

fix(core): Add missing required metadata filters #9543

Merged

Conversation

@PeterStaev
Copy link
Contributor

@PeterStaev PeterStaev commented Sep 11, 2021

PR Checklist

What is the current behavior?

If you create a new app and add the native-api-usage.json in the respective App_Resources folder you will see that the app run fails for both iOS and Android because some used classes were not added in the core whitelist.

What is the new behavior?

This adds the required metadata filters for a new app to run.

Fixes/Implements/Closes #[Issue Number].

@cla-bot
Copy link

@cla-bot cla-bot bot commented Sep 11, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign the CLA at https://www.nativescript.org/cla.
CLA has not been signed by users: @PeterStaev.
After signing the CLA, you can ask me to recheck this PR by posting @cla-bot check as a comment to the PR.

Loading

@PeterStaev
Copy link
Contributor Author

@PeterStaev PeterStaev commented Sep 11, 2021

@cla-bot check

Loading

@cla-bot cla-bot bot added the cla: yes label Sep 11, 2021
@cla-bot
Copy link

@cla-bot cla-bot bot commented Sep 11, 2021

The cla-bot has been summoned, and re-checked this pull request!

Loading

Copy link
Contributor

@NathanWalker NathanWalker left a comment •

Thank you @PeterStaev!
This will go into 8.1.2

Loading

@NathanWalker NathanWalker merged commit a51b628 into NativeScript:master Sep 11, 2021
2 of 3 checks passed
Loading
@PeterStaev PeterStaev deleted the fix/fix-metadata-filters branch Sep 11, 2021
@NathanWalker
Copy link
Contributor

@NathanWalker NathanWalker commented Sep 15, 2021

Loading

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

Successfully merging this pull request may close these issues.

None yet

2 participants