Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: primex adapter #908

Merged
merged 2 commits into from
Oct 20, 2023
Merged

feat: primex adapter #908

merged 2 commits into from
Oct 20, 2023

Conversation

TbIKoBKa
Copy link
Contributor

No description provided.

@llamatester
Copy link

The primex-finance adapter exports:


Dexs for 19/10/2023
_______________________________________



Done in 3.34s.

@TbIKoBKa
Copy link
Contributor Author

start date is 19/10/2023, but no volume results
if set to 18/10/2023 everything works fine

@dtmkeng
Copy link
Contributor

dtmkeng commented Oct 20, 2023

yeah we collect yesterday's data if start time = yesterday's date is not the result

@TbIKoBKa
Copy link
Contributor Author

@dtmkeng so can we merge adapter with yesterday's date or wait until tomorrow?

@dtmkeng
Copy link
Contributor

dtmkeng commented Oct 20, 2023

we can merge today if you okay about result data, could you set start time 18/10/2023 for check result data ?

@llamatester
Copy link

The primex-finance adapter exports:


Dexs for 19/10/2023
_______________________________________

POLYGON 👇
Backfill start time: 18/10/2023
NO METHODOLOGY SPECIFIED
Daily volume: 21476.631201253727
Timestamp: 1697759998




Done in 16.43s.

@TbIKoBKa
Copy link
Contributor Author

@dtmkeng done

@llamatester
Copy link

The primex-finance adapter exports:


Dexs for 19/10/2023
_______________________________________



Done in 3.44s.

@dtmkeng
Copy link
Contributor

dtmkeng commented Oct 20, 2023

this value 1697587200

@llamatester
Copy link

The primex-finance adapter exports:


Dexs for 19/10/2023
_______________________________________

POLYGON 👇
Backfill start time: 18/10/2023
NO METHODOLOGY SPECIFIED
Daily volume: 21476.631201253727
Timestamp: 1697759998




Done in 12.90s.

@TbIKoBKa
Copy link
Contributor Author

@dtmkeng yep, but the final start date should be 19/10/2023

@dtmkeng dtmkeng merged commit 84596d4 into DefiLlama:master Oct 20, 2023
2 checks passed
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