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(indexer): optional configs not returning undefined when not set #65

Merged
merged 1 commit into from
Jul 7, 2024

Conversation

danadi7
Copy link
Collaborator

@danadi7 danadi7 commented Jul 5, 2024

  • Configs can be optional, but setting optional as true but not setting the variable on .env throws an error
  • This fix returns undefined when an optional config is not set

@thedoublejay thedoublejay merged commit 04eafc4 into main Jul 7, 2024
danadi7 added a commit that referenced this pull request Aug 10, 2024
* Fix bun: symbol lookup error (#64)

* Back to runestones-lib upstream

* fix(indexer): optional configs not returning undefined when not set (#65)

* fix(ord): skip missing inscription (#66)

* fix(ord): skip over missing inscriptions from ord

* fix(ord): fix index and error handling on getInscriptions

* Remove conflict messages

* Fix runestones inconsistency:

- added transactions array to indexer and used RawTransaction for
  runestones deciphering

* Add rune indexer necessary collections

* Runes indexer implementation

* Fix bun: symbol lookup error (#64)

* Back to runestones-lib upstream

* Remove conflict messages

* Fix runestones inconsistency:

- added transactions array to indexer and used RawTransaction for
  runestones deciphering

* Add rune indexer necessary collections

* Runes indexer implementation

* Add API endpoints GetBlockIndex, GetRuneIndex and GetBalancesIndex

* Add Runes.GetAddressRuneUTXOIndex endpoint

* Add bigInt to string utils for storing big numbers in mongodb as strings

* Fix NaN blockHeight log

* Fix runes indexer for spent and burned balances

- Reuse db and rpc methods
- Remove duplicated code
- Small tweaks in logs

* GetSpendablesIndex endpoint implementation (#71)

* Add concurrency limit to vout processing

* Fix typo

* Use existing limiter

* Add fix for mgetValidMintCount and fix bug on utxos composite unique key

* Fix balances with big amounts saved as strings in db

* Remove scriptPubKey field from output

- Fix balances add from string to bigint
- Remove duplicated endpoint Runes.GetSpendablesIndex (use
  Address.GetSpendables or Address.GetUnspents)
- Typo

* Move db call outside balances loop

---------

Co-authored-by: Daniel Adi <48890526+danadi7@users.noreply.github.com>
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.

2 participants