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

Db/deprecate groups #198

Merged
merged 6 commits into from
Nov 1, 2023
Merged

Db/deprecate groups #198

merged 6 commits into from
Nov 1, 2023

Conversation

davidbloss
Copy link
Contributor

Issues

#132

Changelog

Deprecate group commands in favor of Team hierarchies.
group create and group update are no-ops. The other group commands direct devs to the docs and to use Teams.

  • List your changes here
  • Make a changie entry

Tophatting

@davidbloss davidbloss requested review from rocktavious and a user October 31, 2023 21:01
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #198 (eb9f976) into main (c954a59) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #198   +/-   ##
=====================================
  Coverage   6.22%   6.22%           
=====================================
  Files          5       5           
  Lines        257     257           
=====================================
  Hits          16      16           
  Misses       241     241           

src/cmd/group.go Outdated
Use: "group",
Aliases: []string{"groups"},
Short: "Imports groups from a CSV",
Deprecated: `Please convert all Groups into Teams. See https://docs.opslevel.com/docs/groups`,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since you removed the create and update functionality - should we also remove the "import" functionality?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm thinking that customers might use this to get a snapshot of their current Groups so that they can know what's left to convert to Teams

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing per offline discussion

Copy link
Collaborator

@rocktavious rocktavious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - just 1 question but otherwise good to ship.

@davidbloss davidbloss merged commit 88b8d91 into main Nov 1, 2023
3 checks passed
@davidbloss davidbloss deleted the db/deprecate-groups branch November 1, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants