Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Use last_verified_at in Singularity as deal updated_at #172

Merged
merged 3 commits into from
Oct 20, 2023

Conversation

xinaxu
Copy link
Collaborator

@xinaxu xinaxu commented Oct 19, 2023

resolves #162
See above issue for context
Uses the correct field in Singularity for deal status

@xinaxu xinaxu requested a review from masih October 19, 2023 18:52
@codecov-commenter
Copy link

Codecov Report

Merging #172 (782030a) into main (7230d07) will increase coverage by 0.67%.
Report is 6 commits behind head on main.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
+ Coverage   30.82%   31.49%   +0.67%     
==========================================
  Files           5        5              
  Lines         743      743              
==========================================
+ Hits          229      234       +5     
+ Misses        479      474       -5     
  Partials       35       35              
Files Coverage Δ
integration/singularity/store.go 36.36% <0.00%> (+1.08%) ⬆️

@xinaxu xinaxu merged commit eb956fc into main Oct 20, 2023
@xinaxu xinaxu deleted the xinaxu/deal-last-verified branch October 20, 2023 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change deal status LastVerified to be based on Singularity LastVerifiedAt
3 participants