-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[spaceship] added b2b and educational discount to app #17490
Conversation
I was able to test this and it works well! I only used the Adding an org worked fine, and enabling B2B worked fine. However I was not able to disable B2B on an app that already had it enabled. I get the error I believe this is because the |
def disable_b2b | ||
update(attributes: { | ||
distributionType: DistributionType::APP_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.
I am getting an error on this because the educational discount type is still NOT_APPLICABLE
, but that is not acceptable when distribution type is 'APP_STORE'
When an app has NOT_APPLICABLE education discount, only CUSTOM app distribution is possible
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.
Ahhhhh.... good catch! I will make that change 😊
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.
This should be fixed now!
class CustomAppOrganization | ||
include Spaceship::ConnectAPI::Model | ||
|
||
attr_accessor :device_enrollment_program_id |
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.
consider more consistent names for these attributes? dep_id
and dep_name
maybe?
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.
This is the name of it in the App Store Connect API model/resource so we've been sticking with those exact same names so it also matches what is (or will be) in the official doc 😇
Looks good! |
Hey @joshdholtz 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment here 👍 |
Congratulations! 🎉 This was released as part of fastlane 2.165.0 🚀 |
Motivation and Context
Fixes #17244
Description
Spaceship::ConnectAPI::CustomAppUser
Spaceship::ConnectAPI::CustomAppUser.create
Spaceship::ConnectAPI::CustomAppUser.all
user.delete!
Spaceship::ConnectAPI::CustomAppOrganization
Spaceship::ConnectAPI::CustomAppOrganization.create
Spaceship::ConnectAPI::CustomAppOrganization.all
organization.delete!
Spaceship::ConnectAPI::App
modelapp.enable_b2b
app.disable_b2b
app.enable_educational_discount
app.disable_educational_discount
Example
Testing Steps
Update
Gemfile
and runbundle install
,bundle update fastlane
, orbundle update