Skip to content

Conversation

seantleonard
Copy link
Contributor

Port metadata

Why make this change?

What is this change?

  • These changes impact API builder startup. The query output has not been modified, but certain field references are capitalized.
  • The proper case is used to avoid "field not found" and "key field not found" errors when the database collation is case-sensitive.

How was this tested?

  • Integration Tests - Ran locally; about 5 tests failed, but it was the same ones that failed before modifications
  • Unit Tests

Sample Request(s)

  • N/A

## Why make this change?

- Resolves #1896

## What is this change?

- These changes impact API builder startup. The query output has not
been modified, but certain field references are capitalized.
- The proper case is used to avoid "field not found" and "key field not
found" errors when the database collation is case-sensitive.

## How was this tested?

- [x] Integration Tests - Ran locally; about 5 tests failed, but it was
the same ones that failed before modifications
- [ ] Unit Tests

## Sample Request(s)
- N/A

---------

Co-authored-by: Sean Leonard <sean.leonard@microsoft.com>
Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>
Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
Co-authored-by: Ruben Cerna <rcernaserna@microsoft.com>
@Aniruddh25
Copy link
Collaborator

/azp run

@seantleonard seantleonard merged commit 58cf603 into release/1.2 Aug 8, 2024
@seantleonard seantleonard deleted the dev/sean/1896_port_to_1.2 branch August 8, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒Cherrypick Cherry-picking another commit/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants