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

[v15] Replace isTeam with new flags #41217

Merged
merged 2 commits into from
May 13, 2024

Conversation

mcbattirola
Copy link
Contributor

Backports #39794 to v15

* Add new flags to modules

* Add deprecated comments to isTeam

* Make support flag an enum

* Add a TODO instead of deprecating flag right away

Co-authored-by: Alan Parra <[email protected]>

* Add gogoproto.jsontag to new fields

* remove meaning from enum zero value

* Reuse proto.SupportType instead of suping

* Replace isTeam for new flags in web app

* Update proto file: rename enum 0 value to unspecified;fix json tag casing

* Reuse proto file instead of aliasing it

* undo removing method by accident

* Add MobileDeviceManagement field to web config struct

* Improve comments

* Fix tests

* Simplify lockedFeatures object

* Include JoinActiveSessions in the web config object

* Remove more instances of isTeam

* Apply suggestions from code review - improve comments

Co-authored-by: Michelle Bergquist <[email protected]>

* Use consistent comments to remove isTeam

* Update godocs comments

* Revert removign `isTeam` from stories it still should

* Fix godoc typo

Co-authored-by: Michelle Bergquist <[email protected]>

---------

Co-authored-by: Alan Parra <[email protected]>
Co-authored-by: Michelle Bergquist <[email protected]>
@mcbattirola mcbattirola added the no-changelog Indicates that a PR does not require a changelog entry label May 6, 2024
@mcbattirola mcbattirola marked this pull request as ready for review May 6, 2024 13:39
@zmb3 zmb3 changed the title [v15] Replace isTeam with new flags (#39794) [v15] Replace isTeam with new flags May 6, 2024
@mcbattirola mcbattirola added this pull request to the merge queue May 13, 2024
Merged via the queue into branch/v15 with commit 9aa2e64 May 13, 2024
40 checks passed
@mcbattirola mcbattirola deleted the mcbattirola/v15/remove-team--use-new-flags branch May 13, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/sm ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants