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

Unnecessary BOND_MAP environment variable #24

Closed
fleupold opened this issue Jul 9, 2024 · 2 comments · Fixed by #29
Closed

Unnecessary BOND_MAP environment variable #24

fleupold opened this issue Jul 9, 2024 · 2 comments · Fixed by #29

Comments

@fleupold
Copy link
Contributor

fleupold commented Jul 9, 2024

const { FEE_QUERY, BILLING_QUERY, PAYMENT_QUERY, DUNE_API_KEY, BOND_MAP } =

We are already defining a payment query in the environment. Can we expect that query to specify the fine/draft target in one of its columns instead of defining another mapping here?

This would simplify the setup.

@cowanator
Copy link
Collaborator

The payment query is just a query ID. The BOND Map is meant to be a query parameter. We can hard-code this mapping in the query we want, but the more correct approach is to use parameters for this (in case it changes). Note that the other queries are currently hard coding this mapping:

Line 107 of https://dune.com/queries/3630322
which relies on this: https://dune.com/queries/3665113

Anyway, if we want to go that route we would need to adjust the PAYMENT_QUERY: https://dune.com/queries/3742749

Notice that the payment query was written and supplied by my account, while the others were provided.

Please let me know how you would like to approach this.

@fleupold
Copy link
Contributor Author

I think we should move it into the query (so that it's co-located with the other logic). Ideally the query is also checked into this repository and automatically synced as part of the CD pipeline (but even without that, having all query related parts in one place is better imo)

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 a pull request may close this issue.

2 participants