feat: add support for list of permission...

[sage meadow]

Hi @knotty cape, I ran the test locally and debugged it. The error happens because the LinkedEntrances array is sorted differently than expected due to concurrency. We can update the test to pass. We're also working on pagination in lookup entity and subject with @lofty oar, so we can merge your PR with those changes.

i will update the PR shortly, also we can track the rest through Pr page on Github

thanks a lot for your time and contribution!

[knotty cape]

ohh , okk thanks got it

[knotty cape]

Hi @sage meadow any update on this , I see pagination changes are merged in master branch

[sage meadow]

Hi @knotty cape, i will merge this week

[knotty cape]

okk thanks , let me know if some change required would be more thn happy to do

[knotty cape]

@sage meadow just bumping the msg if we can pls proceed here

[sage meadow]

Hi @knotty cape, I have updated your pr recently. we will review it and merge today if everything is fine. thank you for your contribution again!

[knotty cape]

okk thanks a lot

[knotty cape]

please let me know also If I need to do anything

[knotty cape]

Hi @sage meadow is there any update pls

[sage meadow]

Hi @knotty cape, i updated the pr and reviewed. but now there are some small conflicts. let me ask tolga and reply to you.

also, we prefer to add new endpoint/api instead of changing one. can you update the pr accordingly?

[knotty cape]

Ohh sure will update on weekend

[knotty cape]

Update :- I have done the changes , But I need to fix few things and also little bit refactor , It was turned out to be a big change so it may take few more days for me

[knotty cape]

Hi @sage meadow I have refactored and fixed most of test cases , all test cases are working for old lookup API but for new API 2 test cases are failing (returning duplicate permission or some extra permission)and I feel its due to race condition , If i pass single permission in list in that case even these are also working, I tried debugging and i feel its in bulk.ExecuteRequests function somewhere. Can you pls take a look into it

[knotty cape]

Helo @sage meadow Did you got a chance to review it ?

[sage meadow]

Hi @knotty cape, yes I took a look at it. I will make some updates this weekend.