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

[CoE Starter Kit - Feature]: Short term patch - update shared with flow to get true count #7738

Closed
Jenefer-Monroe opened this issue Feb 27, 2024 · 2 comments
Assignees
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request

Comments

@Jenefer-Monroe
Copy link
Collaborator

Jenefer-Monroe commented Feb 27, 2024

Is your feature request related to a problem? Please describe.

We will be refactoring this flow soon so that it can run more frequently and be expanded then to other objects.
But in the near term we can patch it to fetch the true count of the group

Describe the solution you'd like

using that HTTP call I found.

Describe alternatives you've considered

No response

Additional context?

No response

AB#2392

@Jenefer-Monroe Jenefer-Monroe added enhancement New feature or request coe-starter-kit CoE Starter Kit issues labels Feb 27, 2024
@Jenefer-Monroe Jenefer-Monroe self-assigned this Feb 27, 2024
@RajeevPentyala RajeevPentyala moved this to Todo ✏️ in CoE Starter Kit Feb 27, 2024
@Jenefer-Monroe
Copy link
Collaborator Author

image

@Jenefer-Monroe
Copy link
Collaborator Author

this does not get us a unique list so if a user is in multiple groups that belong to the app they will be double counted.
it also does not get us if the user is guest, app, etc.
but today's situation is worse in that we dont actually know when things are overshared at all.
So think this is a good change until we do the deeper work to include the above.

@Jenefer-Monroe Jenefer-Monroe moved this from Todo ✏️ to Testing 🤞 in CoE Starter Kit Mar 3, 2024
@Jenefer-Monroe Jenefer-Monroe moved this from Testing 🤞 to Code complete ☑ in CoE Starter Kit Mar 13, 2024
@github-project-automation github-project-automation bot moved this from Code complete ☑ to Done ✅ in CoE Starter Kit Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coe-starter-kit CoE Starter Kit issues enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

2 participants