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

Fix odos_v2_optimism_trades.sql #5404

Merged

Conversation

amalashkevich
Copy link
Contributor

Thank you for contributing to Spellbook!

Thank you for taking the time to submit code in Spellbook. A few things to consider:

  • If you are a first-time contributor, please sign the CLA by copy & pasting exactly what the bot mentions in PR comment
  • Refer to docs section below to answer questions
  • Dune team will review submitted PRs as soon as possible

Spellbook sub-projects

To prepare for Spellbook to scale in size, Dune team has started to pilot sub-projects:

  • For a quick overview, see the GH discussion on the topic
    • Please ask questions and continue the conversation there
  • The first spell lineage to move: tokens.erc20 and all upstream spells which help build it
    • Note: Please navigate to tokens root directory to find tokens objects

Spellbook contribution docs

The docs directory has been implemented to answer as many questions as possible. Please take the time to reference each .md file within this directory to understand how to efficiently contribute & why the repo is designed as it is 🪄

Example questions to be answered:

Please navigate through the docs directory to find as much info as you can.

Note: happy to take PRs to improve the docs, let us know 🤝

@dune-eng
Copy link

Workflow run id 8008981763 approved.

@dune-eng
Copy link

Workflow run id 8008982148 approved.

@dune-eng
Copy link

Workflow run id 8021293237 approved.

@dune-eng
Copy link

Workflow run id 8021293428 approved.

@dune-eng
Copy link

Workflow run id 8032439396 approved.

@dune-eng
Copy link

Workflow run id 8032439448 approved.

@jeff-dude jeff-dude added ready-for-merging and removed ready-for-review this PR development is complete, please review labels Feb 25, 2024
@jeff-dude jeff-dude self-assigned this Feb 25, 2024
@dune-eng
Copy link

Workflow run id 8051067482 approved.

@dune-eng
Copy link

Workflow run id 8051067859 approved.

@amalashkevich
Copy link
Contributor Author

@jeff-dude
Can I do anything to fix this error?

dbt slim ci / dbt-test (spellbook) (pull_request)

15:18:51 Finished running in 0 hours 0 minutes and 0.04 seconds (0.04s).
17
15:18:51 Encountered an error:
18
Runtime Error
19
error 500: b'\n\n\n<title>Error 500 java.lang.IllegalArgumentException: Invalid URI host: null (authority: null)</title>\n\n

HTTP ERROR 500 java.lang.IllegalArgumentException: Invalid URI host: null (authority: null)

\n\n\n\n\n\n\n
URI:/v1/statement
STATUS:500
MESSAGE:java.lang.IllegalArgumentException: Invalid URI host: null (authority: null)
SERVLET:prestoRouter
CAUSED BY:java.lang.IllegalArgumentException: Invalid URI host: null (authority: null)
\n

Caused by:

java.lang.IllegalArgumentException: Invalid URI host: null (authority: null)\n\tat

@dune-eng
Copy link

Workflow run id 8055302813 approved.

@dune-eng
Copy link

Workflow run id 8055302811 approved.

@dune-eng
Copy link

Workflow run id 8059960724 approved.

@dune-eng
Copy link

Workflow run id 8059960806 approved.

@jeff-dude
Copy link
Member

@jeff-dude
Can I do anything to fix this error?

think this was just a one-time cluster error. i believe we got the same thing in prod.

@jeff-dude jeff-dude merged commit d3d2c26 into duneanalytics:main Feb 27, 2024
3 checks passed
Copy link

gitpoap-bot bot commented Feb 27, 2024

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2024 Dune Contributor:

GitPOAP: 2024 Dune Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to handle multiple input and output tokens in a single atomic transactions in dex.trades?
3 participants