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(bun): symbol lookup error #64

Merged
merged 1 commit into from
Jul 7, 2024

Conversation

dcorral
Copy link
Contributor

@dcorral dcorral commented Jul 4, 2024

According to oven-sh/bun#5835 there is not know fix/workaround. We can come back to this if the issue is resolved. Back to using tsx.

Related issue: #52

@dcorral dcorral changed the title Fix bun: symbol lookup error fix(bun): symbol lookup error Jul 4, 2024
Copy link
Collaborator

@danadi7 danadi7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@thedoublejay thedoublejay merged commit 2e7f442 into main Jul 7, 2024
dcorral added a commit that referenced this pull request Jul 15, 2024
dcorral added a commit that referenced this pull request Jul 19, 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.

3 participants