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

VPP > Edit teams: removing a team from a VPP token removes App Store apps from the team #21827

Closed
17 of 19 tasks
noahtalerman opened this issue Sep 4, 2024 · 11 comments
Closed
17 of 19 tasks
Assignees
Labels
~frontend Frontend-related issue. #g-software Software product group :product Product Design department (shows up on 🦢 Drafting board) story A user story defining an entire feature
Milestone

Comments

@noahtalerman
Copy link
Member

noahtalerman commented Sep 4, 2024

Goal

User story
As a IT admin removing a team from a VPP token,
I want to know how this affects my hosts and my App Store apps that are available for install for that team
so that I'm aware of the consequences before I remove the team.

Objective

None. This is cleanup for the "Add multiple ABM and VPP tokens" story here: #9956

More context: #21804 (comment)

Original requests

None.

Context

Changes

Product

  • UI changes: Figma wireframes here
  • CLI (fleetctl) usage changes: No changes
  • YAML changes: No changes
  • REST API changes: No changes
  • Fleet's agent (fleetd) changes: No changes
  • Activity changes: No changes
  • Permissions changes: No changes
  • Changes to paid features or tiers: No changes
  • GitOps mode changes: No changes
  • Transparency changes: No changes
  • Other reference documentation changes: No changes
  • First draft of test plan added: @noahtalerman: No drafted test plan. We worked on this story before the new user story review ritual. Followed up about QA here: VPP > Edit teams: removing a team from a VPP token removes App Store apps from the team #21827 (comment)
  • Once shipped, requester has been notified
  • Once shipped, dogfooding issue has been filed: @noahtalerman: No dogfood issue needed. Confirmed the copy is updated in dogfood

Engineering

  • Feature guide changes: None
  • Database schema migrations: None
  • Load testing: None

ℹ️  Please read this issue carefully and understand it. Pay special attention to UI wireframes, especially "dev notes".

QA

Risk assessment

  • Requires load testing: None
  • Risk level: Low / High None
  • Risk description: None

Manual testing steps

  1. Load modal and verify copy update

Testing notes

Confirmation

  1. Engineer (@____): Added comment to user story confirming successful completion of QA.
  2. QA (@____): Added comment to user story confirming successful completion of QA.
@noahtalerman noahtalerman added :product Product Design department (shows up on 🦢 Drafting board) ~feature fest Will be reviewed at next Feature Fest labels Sep 4, 2024
@noahtalerman noahtalerman self-assigned this Sep 5, 2024
@noahtalerman noahtalerman added story A user story defining an entire feature #g-mdm MDM product group and removed ~feature fest Will be reviewed at next Feature Fest labels Sep 5, 2024
@noahtalerman noahtalerman removed their assignment Sep 12, 2024
@noahtalerman noahtalerman added ~feature fest Will be reviewed at next Feature Fest and removed :product Product Design department (shows up on 🦢 Drafting board) labels Sep 12, 2024
@noahtalerman
Copy link
Member Author

This story didn't make the 3 week drafting timeline. We removed it from the drafting board and added it to feature fest.

@noahtalerman noahtalerman added :product Product Design department (shows up on 🦢 Drafting board) and removed ~feature fest Will be reviewed at next Feature Fest :product Product Design department (shows up on 🦢 Drafting board) labels Sep 18, 2024
@noahtalerman noahtalerman self-assigned this Sep 18, 2024
@noahtalerman noahtalerman added ~feature fest Will be reviewed at next Feature Fest and removed ~feature fest Will be reviewed at next Feature Fest labels Oct 3, 2024
@noahtalerman noahtalerman added ~feature fest Will be reviewed at next Feature Fest and removed ~feature fest Will be reviewed at next Feature Fest labels Oct 24, 2024
@noahtalerman noahtalerman changed the title VPP > Edit teams: copy for changing a team's VPP token means you'll need to re-add apps to the hosts VPP > Edit teams: removing a team from a VPP token means you'll need to re-add apps to the team Nov 6, 2024
@noahtalerman noahtalerman changed the title VPP > Edit teams: removing a team from a VPP token means you'll need to re-add apps to the team VPP > Edit teams: removing a team from a VPP token removes App Store apps from the team Nov 6, 2024
@noahtalerman noahtalerman added :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. and removed :product Product Design department (shows up on 🦢 Drafting board) labels Nov 15, 2024
@lukeheath lukeheath added this to the 4.60.0 milestone Nov 15, 2024
@georgekarrv georgekarrv modified the milestones: 4.60.0, 4.61.0-tentative Nov 21, 2024
@lukeheath lukeheath added the ~frontend Frontend-related issue. label Nov 25, 2024
@lukeheath lukeheath assigned ghernandez345 and unassigned lukeheath Dec 10, 2024
@lukeheath
Copy link
Member

@ghernandez345 I thought I could knock this out, but I don't have the MDM environment setup and haven't gotten around to adding it. Handing this off to you so I don't continue to block it.

@georgekarrv georgekarrv modified the milestones: 4.61.0, 4.62.0-tentative Dec 17, 2024
@lukeheath lukeheath added the #g-software Software product group label Dec 19, 2024
@mostlikelee mostlikelee removed the #g-mdm MDM product group label Jan 2, 2025
@RachelElysia RachelElysia modified the milestones: 4.62.0, 4.63.0-tentative Jan 2, 2025
@jmwatts
Copy link
Member

jmwatts commented Jan 14, 2025

QA Notes

  • Verified edit modal for existing VPP token now states "If you remove a team, the App Store apps will be removed from that team. They won't be uninstalled from hosts."

@lukeheath lukeheath added :product Product Design department (shows up on 🦢 Drafting board) and removed :release Ready to write code. Scheduled in a release. See "Making changes" in handbook. labels Feb 4, 2025
@noahtalerman
Copy link
Member Author

Image

@jmwatts just checking, did we test that App Store apps and their policies are actually deleted when I remove a team?

@jmwatts
Copy link
Member

jmwatts commented Feb 5, 2025

@noahtalerman This was a front-end only change. The testing note is on the original ticket here: #21804 (comment)

@noahtalerman
Copy link
Member Author

@noahtalerman This was a front-end only change. The testing note is on the original ticket here: #21804 (comment)

@jmwatts thanks!

@PezHub did we test if App Store app's policies are removed from a team when that team is unassigned from VPP token? (in addition to the App Store apps themselves being removed)

@jmwatts
Copy link
Member

jmwatts commented Feb 6, 2025

FWIW yesterday I was doing some testing in this area and the policies are not removed, but the app install automation was removed from the policy.

@PezHub
Copy link
Contributor

PezHub commented Feb 7, 2025

Thanks Janis! I just walked thru the workflow to confirm @noahtalerman

@noahtalerman
Copy link
Member Author

noahtalerman commented Feb 7, 2025

Thanks! During design review we decided to update this copy so that it's accurately matches Fleet's current behavior:

Image

I opened a PR here: #26166

@fleet-release
Copy link
Contributor

Removing teams, a shift,
VPP tokens adapt, drift,
Apps stay aloft, swift.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
~frontend Frontend-related issue. #g-software Software product group :product Product Design department (shows up on 🦢 Drafting board) story A user story defining an entire feature
Projects
None yet
Development

No branches or pull requests

9 participants