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 - BUG] Manage Permissions App -- too many rows to count causes failure to load #7841

Closed
1 task done
nasilvae opened this issue Mar 13, 2024 · 17 comments
Closed
1 task done
Assignees
Labels
bug Something isn't working coe-starter-kit CoE Starter Kit issues

Comments

@nasilvae
Copy link

nasilvae commented Mar 13, 2024

Does this bug already exist in our backlog?

  • I have checked and confirm this is a new bug.

Describe the issue

Error when trying to retrieve data from the network

image

Expected Behavior

App should run

What solution are you experiencing the issue with?

Core

What solution version are you using?

4.24

What app or flow are you having the issue with?

Manage Permissions App

What method are you using to get inventory and telemetry?

Data Export

Steps To Reproduce

Open App
App Shows Empty

Anything else?

No response

AB#2558

@nasilvae nasilvae added bug Something isn't working coe-starter-kit CoE Starter Kit issues labels Mar 13, 2024
@RajeevPentyala RajeevPentyala moved this to Todo ✏️ in CoE Starter Kit Mar 13, 2024
@Jenefer-Monroe
Copy link
Collaborator

What state are you in here? Have you run the setup wizard already in this envt?
I dont think you are seeng that due to an empty table.

@Jenefer-Monroe
Copy link
Collaborator

Here's the table it reads from I presume it already has data as its not a new table: 'Environment Security Role Permissions'

@Jenefer-Monroe Jenefer-Monroe self-assigned this Mar 13, 2024
@nasilvae
Copy link
Author

Its on my test tenant. I ran it on both environments and I am able to repro on both. One has the data export another has the cloud flow version.

image

@nasilvae
Copy link
Author

Here's the table it reads from I presume it already has data as its not a new table: 'Environment Security Role Permissions'

image

@Jenefer-Monroe
Copy link
Collaborator

I don’t repro so not sure how you got this
Can you open in edit mode and see where it’s failing?

@nasilvae
Copy link
Author

I don’t repro so not sure how you got this Can you open in edit mode and see where it’s failing?

I found the issue. If I filter the table then I don't get the error.

image.

I suspect that it has to do with how many items can be displayed at one time in the power app. I have over 60k records on this table that should be shown all at the time in the app.
image

We need a way to paginate the information for large tenants.

@Jenefer-Monroe
Copy link
Collaborator

Count rows should be delegatable so I dont think thats it. What filter did you use? Maybe there is some malformed data?

@Jenefer-Monroe
Copy link
Collaborator

I'm trying an upgrade from Feb to see if its something you experience before inventory cleans up / adds new stuff to the table.

@nasilvae
Copy link
Author

I'm trying an upgrade from Feb to see if its something you experience before inventory cleans up / adds new stuff to the table.

Gotcha, so far on my side the following filters work
image
image
image
image
image

If I filter to All then I get 4 error messages
image

@Jenefer-Monroe
Copy link
Collaborator

Upgrade theory failed.

Can you go to the developer tools (in edge browse settings > more tools > developer tools) and then find the network tab
Filter to Guest, wait till it resolves, and then filter back to All
Do you see an error in the network trace?
image

@AUTigers16
Copy link

I am experiencing the same issue. However, I cannot filter the list to be able to get data to appear like the person reporting the bug. It was working fine before I ran this flow, Admin | Gather Tenant SRs, which I think pulls in all permissions in our tenant besides the Security Administrator role. Since I still can't post images here, I will email them to you and reference the bug. I edited the app and grabbed screenshots of where I was getting errors.

@nasilvae
Copy link
Author

Upgrade theory failed.

Can you go to the developer tools (in edge browse settings > more tools > developer tools) and then find the network tab Filter to Guest, wait till it resolves, and then filter back to All Do you see an error in the network trace? image

found the error in the network trace. Will be sending you the file separately.
image

@Jenefer-Monroe
Copy link
Collaborator

thanks guys. Looks like count isnt delegating after all and is failing for you. I can put some fake data in to see if I can repro

@Jenefer-Monroe
Copy link
Collaborator

Well thats an error I didnt know existed
image

@Jenefer-Monroe
Copy link
Collaborator

Looks like CountRows isnt delegating after all. Which it is supposed to be supported for Dataverse. But let me try to find a workaround.

@Jenefer-Monroe
Copy link
Collaborator

looks like there isnt a solution to getting count for tables this big without using middleware.
But its just for the pagination number so here is what you will see for your large cases now:
image

@Jenefer-Monroe Jenefer-Monroe moved this from Todo ✏️ to Code complete ☑ in CoE Starter Kit Mar 21, 2024
@Jenefer-Monroe Jenefer-Monroe changed the title [CoE Starter Kit - BUG] Manage Permissions App [CoE Starter Kit - BUG] Manage Permissions App -- too many rows to count causes failure to load Mar 21, 2024
@nasilvae
Copy link
Author

looks like there isnt a solution to getting count for tables this big without using middleware. But its just for the pagination number so here is what you will see for your large cases now: image

This is good thank you. We can load the list but not get counts I believe is fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working coe-starter-kit CoE Starter Kit issues
Projects
Status: Done
Development

No branches or pull requests

4 participants