Skip to content

Conversation

@M4Al
Copy link
Owner

@M4Al M4Al commented Jun 26, 2025

Why make this change?

  • Reference associated issue using # syntax. e.g. Closes #XX
    • Include summary (1-2 sentences) of linked issue to avoid redirecting reviewers to different pages.
    • Include links to any additional discussion threads related to this change.

What is this change?

  • Summary of how your changes work to give reviewers context of your intent.
    • Links to relevant documentation / StackOverflow, if applicable.

How was this tested?

  • Integration Tests
  • Unit Tests

Sample Request(s)

  • Example REST and/or GraphQL request to demonstrate modifications
  • Example of CLI usage to demonstrate modifications

Mu4all and others added 30 commits August 16, 2024 11:41
Added a new constant `ORIGINAL_ROLE_CLAIM_TYPE` in `AuthenticationOptions.cs` to store the original roles claim type. Modified `AuthorizationResolver` to preserve the original 'roles' claim by adding it to the `resolvedClaims` dictionary under the new key. Changed `MsSqlQueryExecutor` to set session context parameters with `@read_only = 0` to allow modifications.
This reverts commit 08f741c.
remove trailing space

Co-authored-by: Aniruddh Munde <[email protected]>
Updated the constant `FIRST_URL` in `RequestParser.cs` within the
`Azure.DataApiBuilder.Core.Parsers` namespace to use the value `"$top"`
instead of `"$first"`. This change aligns with naming conventions or
standards used elsewhere in the codebase or API, ensuring consistency
and improving clarity for developers.
The Offset method now includes a try-catch block to handle
potential HotChocolate.GraphQLException exceptions, ensuring
that the method returns 0 in such cases. This adds robustness
to the method by preventing it from failing unexpectedly when
the "offset" argument cannot be retrieved.
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.

3 participants