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

Improve native app builds #10

Merged
merged 3 commits into from
Jan 19, 2020
Merged

Improve native app builds #10

merged 3 commits into from
Jan 19, 2020

Conversation

ricoberger
Copy link
Member

  • For the native apps the versions are adjusted automatically now.
  • For the first version of kubenav we received a message from Apple with the following title: ITMS-90078: Missing Push Notification Entitlement. To fix this issue we implemented the solution from the Capacitor Documentation
  • We tried to use the Editor component to add clusters via kubeconfig. Because the underlying Ace Editor doesn't support copy and past on iOS we switched back to IonTextarea (see Paste in iOS not working ajaxorg/ace#2970).

On a new release the iOS and Android versions are adjusted automatically
now.
For the last release we received a message from Apple with the following
tile:

  "ITMS-90078: Missing Push Notification Entitlement"

The fix for this is described in the Capacitor documentation:

  https://capacitor.ionicframework.com/docs/apis/push-notifications/
On mobile we are using the IonTextarea component instead of the Editor
component to add a cluster via kubeconfig. This change is necessary
because the underlying Ace Editor does not support copy and paste on
iOS.
@ricoberger ricoberger added the bug Something isn't working label Jan 19, 2020
@ricoberger ricoberger merged commit 8c5a67a into master Jan 19, 2020
@ricoberger ricoberger deleted the improve-native-app-builds branch January 19, 2020 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant