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: add floki-trading-bot dimension adapter #1927

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JCollinsX
Copy link

No description provided.

@llamatester
Copy link

The floki-trading-bot.ts adapter exports:

> adapters@1.0.0 test
> ts-node --transpile-only cli/testAdapter.ts fees floki-trading-bot.ts

🦙 Running FLOKI-TRADING-BOT.TS adapter 🦙
---------------------------------------------------
Start Date:	Wed, 18 Sep 2024 08:06:30 GMT
End Date:	Thu, 19 Sep 2024 08:06:30 GMT
---------------------------------------------------

[Error: Request failed with status code 401] {
  url: 'https://api.dune.com/api/v1/query/4082749/execute',
  method: 'POST',
  axiosError: 'invalid API Key',
  chain: 'bsc'
}

fees/floki-trading-bot.ts Outdated Show resolved Hide resolved
@llamatester
Copy link

The floki-trading-bot.ts adapter exports:

> adapters@1.0.0 test
> ts-node --transpile-only cli/testAdapter.ts fees floki-trading-bot.ts

🦙 Running FLOKI-TRADING-BOT.TS adapter 🦙
---------------------------------------------------
Start Date:	Mon, 23 Sep 2024 19:55:29 GMT
End Date:	Tue, 24 Sep 2024 19:55:29 GMT
---------------------------------------------------

[Error: Request failed with status code 401] {
  url: 'https://api.dune.com/api/v1/query/3996608/execute',
  method: 'POST',
  axiosError: 'invalid API Key',
  chain: 'ethereum'
}

@g1nt0ki g1nt0ki self-requested a review October 9, 2024 09:44
@g1nt0ki g1nt0ki self-assigned this Oct 9, 2024
@g1nt0ki
Copy link
Member

g1nt0ki commented Oct 9, 2024

@JCollinsX can you update the adapter to instead rely on event logs and compute data onchain? if you share contract/event details, I can implement it

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.

4 participants