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

feat(sf): deploy analytics toolbox native app #454

Merged

Conversation

vdelacruzb
Copy link
Contributor

@vdelacruzb vdelacruzb commented Dec 15, 2023

Description

Shortcut

This PR includes the infrastructure to deploy automatically Native Apps versions. Contains some logic to, depending on which kind of version change we have deploy a whole new version, patch, delete previous versions to not surpass the 2 versions limits. Notice that the uploaded versions are called v<major>(v1, v2...) and then we include the whole version in the label so it will be still visible by the customers.

Also will deploy a native app on each merge in order to check that the upgrade keeps doing successfully, as it could crash if anyone use a Temp table in a procedure for instance.

I included a simple README even if this package is not focused on production.

Type of change

  • Feature

Acceptance

I've run some test regarding deployment, re-deployment, updating versions, seeing which functions are available: https://app.shortcut.com/cartoteam/story/367433/native-apps-installation-qa

@vdelacruzb vdelacruzb requested a review from a team as a code owner December 15, 2023 15:31
Copy link

This pull request has been linked to Shortcut Story #353844: Build Analytics Toolbox Native App Package.

@vdelacruzb vdelacruzb requested a review from Jesus89 December 15, 2023 16:09
Copy link
Member

@Jesus89 Jesus89 left a comment

Choose a reason for hiding this comment

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

Great job!

@Jesus89 Jesus89 merged commit e5160fd into main Dec 19, 2023
10 checks passed
@Jesus89 Jesus89 deleted the feature/sc-353844/build-analytics-toolbox-native-app-package branch December 19, 2023 18:40
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.

3 participants