-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#451] Upload the IPA of DesignToolbox to the publication store #452
base: develop
Are you sure you want to change the base?
[#451] Upload the IPA of DesignToolbox to the publication store #452
Conversation
@@ -34,6 +34,10 @@ APPLE_KEY_CONTENT = ENV["OUDS_APPLE_KEY_CONTENT"] | |||
# Bundle identifier of the app | |||
DEVELOPER_APP_IDENTIFIER = ENV["OUDS_DEVELOPER_BUNDLE_IDENTIFIER"] | |||
|
|||
# Upload To Store | |||
# --------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La ligne 38 ne sert à rien, tu peux la retirer :)
|
||
publish_mattermost_notification("📦 ✅ @here The upload to store has been done successfully (version = '#{version}', build number = '#{build_number}')") | ||
rescue => error | ||
publish_mattermost_notification("📦 🚨 @channel Some issue occurred during the upload step (:upload_2_store)") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faudrait indenter ce qu'il y a dans rescue, à savoir les lignes 458 et 459
# PRIVATE LANE UPLOAD TO STORE | ||
# ---------------------------- | ||
desc "PRIVATE LANE UPLOAD TO STORE" | ||
private_lane :upload_2_store do |params| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plutôt upload_to_store
non ?
# --------------------------------------------------------------------------- | ||
desc "PRIVATE LANE UPLOAD TO TESTFLIGHT" | ||
private_lane :upload do |params| | ||
private_lane :upload_2_testflight do |params| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plutôt upload_to_testflight
non ?
build_number = get_build_number(xcodeproj: OUDS_PROJECT) | ||
puts "ℹ️ App build number: '#{build_number}'" | ||
|
||
sh "curl -F 'file=@build/oudsApp.xcarchive' '#{UPLOAD_STORE_URL}'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On est sûr que l'on a bien quelque chose à "file=@build/oudsApp.xcarchive" ?
Note: Please transform
- [ ]
into- (NA)
in the description when things are not applicableRelated issues
#451
Description
Add a curl command in the upload process. This is done in a specific function.
Motivation & Context
Types of change
Checklist
Contribution
Accessibility
Design
Development
Documentation
Checklist (for Core Team only)