Skip to content

Resolve GrantsForUser queries via resolveQuery() #5836

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

Open
wants to merge 4 commits into
base: llb-normalized-grants
Choose a base branch
from

Conversation

dkanney
Copy link
Collaborator

@dkanney dkanney commented Jun 11, 2025

No description provided.

@dkanney dkanney self-assigned this Jun 11, 2025
@dkanney dkanney force-pushed the dkanney-utilize-array-agg-over-ctes branch from b678638 to 0e0a021 Compare June 12, 2025 18:09
@dkanney dkanney force-pushed the dkanney-grants-for-user-refactor-to-resolve-query branch from b0c011a to 26faa6f Compare June 12, 2025 19:13
@dkanney dkanney force-pushed the dkanney-utilize-array-agg-over-ctes branch from 0e0a021 to ffa336f Compare June 12, 2025 23:15
Base automatically changed from dkanney-utilize-array-agg-over-ctes to llb-normalized-grants June 13, 2025 00:06
@dkanney dkanney force-pushed the dkanney-grants-for-user-refactor-to-resolve-query branch from 26faa6f to 6c20b8d Compare June 13, 2025 01:23
@dkanney dkanney force-pushed the dkanney-grants-for-user-refactor-to-resolve-query branch 3 times, most recently from e4ac1f6 to 68cd6c1 Compare June 13, 2025 05:24
@dkanney dkanney force-pushed the dkanney-grants-for-user-refactor-to-resolve-query branch from 68cd6c1 to 0198363 Compare June 13, 2025 05:29
- Remove grantsForUser sub-functions
- Resolve grantsForUser queries via resolveQuery()
- resolveQuery() and map data to GrantTuples in GrantsForUser()
@dkanney dkanney force-pushed the dkanney-grants-for-user-refactor-to-resolve-query branch from 0198363 to 307c429 Compare June 13, 2025 06:25
@dkanney dkanney marked this pull request as ready for review June 13, 2025 06:38
@dkanney dkanney requested a review from a team as a code owner June 13, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant