fix: Make OwnerSchema username nullable#4020
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4020 +/- ##
=======================================
Coverage 98.74% 98.74%
=======================================
Files 831 831
Lines 15181 15181
Branches 4373 4365 -8
=======================================
Hits 14990 14990
Misses 184 184
Partials 7 7
Continue to review full report in Codecov by Harness.
|
Bundle ReportChanges will increase total bundle size by 22 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: gazebo-production-esmAssets Changed:
Files in
view changes for bundle: gazebo-production-systemAssets Changed:
Files in
|
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #4020 +/- ##
=======================================
Coverage 98.74% 98.74%
=======================================
Files 831 831
Lines 15181 15181
Branches 4365 4365
=======================================
Hits 14990 14990
Misses 184 184
Partials 7 7
Continue to review full report in Codecov by Sentry.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #4020 +/- ##
=======================================
Coverage 98.74% 98.74%
=======================================
Files 831 831
Lines 15181 15181
Branches 4373 4365 -8
=======================================
Hits 14990 14990
Misses 184 184
Partials 7 7
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
✅ Deploy preview for gazebo ready!Previews expire after 1 month automatically.
|
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #4020 +/- ##
=======================================
Coverage 98.74% 98.74%
=======================================
Files 831 831
Lines 15181 15181
Branches 4365 4365
=======================================
Hits 14990 14990
Misses 184 184
Partials 7 7
Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
1 participant
Description
This PR addresses a
ZodErrorthat occurred when parsing the response from the/internal/userAPI. The backend was returningnullforowners[0].usernamefor certain Bitbucket users, but the frontend'sOwnerSchema(part ofInternalUserSchema) expectedusernameto be a non-nullable string.This change updates the
OwnerSchemainsrc/services/user/useInternalUser.tsto explicitly allowusernameto benull, aligning the frontend schema with the actual data returned by the backend.Code Example
Notable Changes
Updated
OwnerSchemainsrc/services/user/useInternalUser.tsto allowusernameto benull.Screenshots
Link to Sample Entry
Legal Boilerplate
Look, I get it. The entity doing business as "Codecov" is owned by Harness, Inc. In 2026 Harness acquired Codecov and as a result Harness is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Harness can use, modify, copy, and redistribute my contributions, under Harness's choice of terms.
Fixes GAZEBO-181T