Skip to content
This repository has been archived by the owner on Jun 3, 2022. It is now read-only.

feat(indexer): Indexing Oracles Data #853

Merged
merged 9 commits into from
Mar 16, 2022

Conversation

nichellekoh
Copy link
Contributor

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Reverted Changes in PR #749
Update time intervals according to oracles feed and requirements - 15 mins, 1 hour, 1 day

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

siradji and others added 5 commits March 7, 2022 14:21
Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>
Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>
Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>
Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>
@codeclimate
Copy link

codeclimate bot commented Mar 15, 2022

Code Climate has analyzed commit c69b772 and detected 7 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Duplication 5

View more on Code Climate.

@codecov
Copy link

codecov bot commented Mar 15, 2022

Codecov Report

Merging #853 (c69b772) into main (6f9f1bd) will increase coverage by 0.59%.
The diff coverage is 96.70%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #853      +/-   ##
==========================================
+ Coverage   91.87%   92.46%   +0.59%     
==========================================
  Files         125      127       +2     
  Lines        3469     3675     +206     
  Branches      435      461      +26     
==========================================
+ Hits         3187     3398     +211     
+ Misses        272      267       -5     
  Partials       10       10              
Impacted Files Coverage Δ
src/module.api/price.controller.ts 91.66% <85.71%> (+9.05%) ⬆️
...ule.indexer/model/dftx/set.oracle.data.interval.ts 96.49% <96.49%> (ø)
packages/whale-api-client/src/api/prices.ts 100.00% <100.00%> (ø)
src/module.indexer/model/dftx.indexer.ts 96.61% <100.00%> (+0.05%) ⬆️
src/module.indexer/model/dftx/_module.ts 100.00% <100.00%> (ø)
src/module.model/_module.ts 100.00% <100.00%> (ø)
...c/module.model/oracle.price.aggregated.interval.ts 100.00% <100.00%> (ø)
src/module.api/_module.ts 94.28% <0.00%> (ø)
packages/whale-api-client/src/api/poolpairs.ts 100.00% <0.00%> (ø)
src/module.api/token.controller.ts 97.29% <0.00%> (+0.07%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f9f1bd...c69b772. Read the comment docs.

@fuxingloh fuxingloh merged commit c5821a8 into main Mar 16, 2022
@fuxingloh fuxingloh deleted the nichellekoh/oracle-data-interval branch March 16, 2022 05:11
eli-lim pushed a commit to BirthdayResearch/jellyfishsdk that referenced this pull request Mar 28, 2022
* Revert Changes

Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>

* Removed package installation files

Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>

* package-lock.json

Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>

* Removed unrelated changes

Signed-off-by: Suraj Auwal <siradjiawoual@gmail.com>

* Updated time interval

* Modify Oracle Feed Intervals

* Update feed intervals to 15 mins for testing

* Resolve issue with time bucketing and median time

* Added more test asserts for oracles and prices test

Co-authored-by: Suraj Auwal <siradjiawoual@gmail.com>
Co-authored-by: Joel-David Wong <git@joeldavid.dev>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants