fix: Cast UUID to str before saving in the db #56
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
trent-codecov
approved these changes
Aug 1, 2023
scott-codecov
force-pushed
the
scott/fix-uuid-serialization
branch
from
39a35f7
to
174376a
Compare
Codecov Report
@@ Coverage Diff @@
## main #56 +/- ##
=====================================
Coverage 95.17 95.17
=====================================
Files 572 572
Lines 14365 14365
=====================================
Hits 13671 13671
Misses 694 694
Flags with carried forward coverage won't be shown. Click here to find out more.
|
adrian-codecov
pushed a commit
that referenced
this pull request
Aug 2, 2023
adrian-codecov
added a commit
that referenced
this pull request
Aug 10, 2023
* Add trial logic * Get rid of unecessary comments + linting * reintroduce print statement and revert gql nullability * Added expiration value when expiring trial * Add pretrial user count field * create a new endpoint that marks the completion step for uploads, which triggers notifs and syncs pulls (#49) * fix: Cast UUID to str before saving in the db (#56) * chore: add debug log for files array saving (#47) In an effort to understand why we're getting `SessionArray` data from GCS without 'meta' info we'll add a temporary log line when saving data from the API. Mostly because I don't think this should happen, so I don't expect to see that line. But who knows... Part of codecov/engineering-team#119 * chore: change logic to write data to storage (#48) After moving `ReportDetails.files_array` and `Commit.report` to archive successfilly for Codecov we want to do the same for some selected customers (in `repo_ids`) and then we will want to start the process for new fields. Only for codecov. The way things were before, if you have a field set to write to GCS it is valid for all codecov repos AND all repos under `repo_ids`, while we would only want that for codecov repos. So by changing the `master_write_switch` to strings we can have more power over each individual field. Now we can use: * `True` or `codecov_access` to write *only* for codecov repos * `restricted_access` to write to all repos in `repo_ids` (+ codecov) * `general_access` to write to all repos Because currently we're only writing for codecov repos the change will not affect any customers Part of codecov/engineering-team#100 * fix: Force owner login during pagination test (#58) * merge * Small fixes + addons * Misc additions to the plan service + trial * fix: Ensure we don't accidentally leak stack trace info (#59) * Add pretrial_plan resolver * Fix test --------- Co-authored-by: Dana Yaish <98817625+dana-yaish@users.noreply.github.com> Co-authored-by: scott-codecov <scott@codecov.io> Co-authored-by: Giovanni M Guidini <99758426+giovanni-guidini@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.