-
Notifications
You must be signed in to change notification settings - Fork 483
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
App Store (VPP) apps: create policies for automatic install #23744
Comments
Hey @eugkuo, I chatted w/ Mike McNeil and he left us this feedback on the current Software > Add software UX. I think let's address this feedback while we're working on this user story:
![]() |
Comments from @jmwatts : @jmwatts thought: I might suggest defaulting to “Custom” for the Target because if you’re automatically installing you probably want to make a conscious decision to scope it to specific devices only because this could install paid apps on devices with no way to pull the licenses if that is in error… just a thought @jmwatts thought: This will need to be tested with paid apps and make sure that any macOS apps that are installed are actually using the license, not just installing a free app that doesn’t need a license. Also what happens if we have an automatic install policy but we don’t have enough licenses? Does it still install the app without the license? Does it fail to install the app? @jmwatts thought: Probably late to be having this thought but does it make sense to handle VPP apps the same way as other packages, especially when:
@jmwatts Tim asked if ‘Self service’ is an install method and I think it is, but you CAN edit whether or not it’s available in Self service after saving, you can NOT edit automatically/manual after saving. |
FYI @eugkuo, Marko started some designs for the simplified add software experience here: https://www.figma.com/design/L2KLDw5WzIHRCvHT0T6RWd/%2323118-Fleet-maintained-apps-for-Windows?node-id=21-22 |
I've updated this ticket with designs at the various breakpoints:
@mostlikelee Could you check to see whether the REST API information is correct in the ticket? If so, I'm going to move this to User story review. Thanks! FYI @noahtalerman |
DONE @noahtalerman: Add the "Show schema" button to wireframes |
@mostlikelee just a reminder that we want to prioritize this user story in the next sprint. Can you please complete the TODOs in the "Engineering" section so that we can estimate with the team during #g-software sprint kickoff? |
|
@mostlikelee is this accurate? @iansltx just pulled me into a chat because we're worried this is a wire getting crossed |
@RachelElysia @iansltx great catch! that was a comment for the wrong story, please disregard! |
…as added_at For #23744. TODO: * Test updates * Query to pull more accurate timestamps from activity feed
@jmwatts Added the following to the test plan, corresponding to how I tested #26442:
Also added this for FMAs:
|
Note to self: need to add #26442 (comment) back, but with a query that actually works for no-team |
@jmwatts Added coverage of the optional manual query (see changes file) in the test plan. |
…as added_at (#26442) For #23744. Splitting into another PR: query to pull more accurate timestamps from activity feed # Checklist for submitter If some of the following don't apply, delete the relevant line. <!-- Note that API documentation changes are now addressed by the product design team. --> - [x] Changes file added for user-visible changes in `changes/`, `orbit/changes/` or `ee/fleetd-chrome/changes`. See [Changes files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/Committing-Changes.md#changes-files) for more information. - [x] Input data is properly validated, `SELECT *` is avoided, SQL injection is prevented (using placeholders for values in statements) - [x] If database migrations are included, checked table schema to confirm autoupdate - For database migrations: - [x] Checked schema for all modified table for columns that will auto-update timestamps during migration. - [x] Confirmed that updating the timestamps is acceptable, and will not cause unwanted side effects. - [x] Ensured the correct collation is explicitly set for character columns (`COLLATE utf8mb4_unicode_ci`). - [x] Added/updated automated tests - [x] A detailed QA plan exists on the associated ticket (if it isn't there, work with the product group's QA engineer to add it) - [x] Manual QA for all new/changed functionality
Goal
Key result
Fleer users can automatically install any software in Fleet w/o writing policies.
Original requests
Context
Changes
Product
Engineering
QA
Risk assessment
Test plan
<App store app name>
" is automatically created.API Testing
self_service
set totrue
and verify that you see an easy to understand error message.VPP creation timestamp testing
created_at
timestamp, corresponding to when they were added to a teamcreated_at
timestamp, such that it's different from whatever is in thevpp_apps
tableTest with #25514 for non-happy-path error states
Testing notes
#24609 and #24989 are related and testing efforts should be made to test these together.
Am I able to add apps with automatic install via API or GitOps yet? Or is this UI only?
What happens if I change the teams on the VPP Token or delete it? Do the Policies also get deleted?
Confirmation
The text was updated successfully, but these errors were encountered: