[PR #148] [MERGED] Fix: index with bigints #128

New issue

Closed

opened 2026-05-23 08:31:24 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-23 08:31:24 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/appy-one/acebase/pull/148
Author: @appy-one
Created: 9/5/2022
Status: ✅ Merged
Merged: 9/5/2022
Merged by: @appy-one

Base: master ← Head: fix/index-with-bigints

📝 Commits (7)

62bfabd Fix BigInts not being indexed
7673d00 Fix caching of index queries with BigInts
7dcd4d1 Removed unused imports, +VALUE_TYPES import
8b56d64 Added BigInt index query (+cache) unit tests
0dd7006 updated core dependency
1de236e chore: ESLint fixes
9b08026 chore: build

📊 Changes

8 files changed (+94 additions, -44 deletions)

View changed files

📝 dist/browser.js (+2 -2)
📝 dist/browser.min.js (+1 -1)
📝 package.json (+1 -1)
📝 spec/indexes.spec.js (+20 -20)
📝 spec/query.spec.js (+53 -1)
📝 src/data-index/index.js (+7 -8)
📝 src/data-index/index.js.map (+1 -1)
📝 src/ts/data-index/index.ts (+9 -10)

📄 Description

Fixes BigInts not being indexed
Fixes #141 (caching of queries with BigInts in filters/take/skip)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/appy-one/acebase/pull/148 **Author:** [@appy-one](https://github.com/appy-one) **Created:** 9/5/2022 **Status:** ✅ Merged **Merged:** 9/5/2022 **Merged by:** [@appy-one](https://github.com/appy-one) **Base:** `master` ← **Head:** `fix/index-with-bigints` --- ### 📝 Commits (7) - [`62bfabd`](https://github.com/appy-one/acebase/commit/62bfabd9fa6b4f3dd2288bb905119314be0de570) Fix BigInts not being indexed - [`7673d00`](https://github.com/appy-one/acebase/commit/7673d0089aa0cdc4fa470412625c7af1613f4f21) Fix caching of index queries with BigInts - [`7dcd4d1`](https://github.com/appy-one/acebase/commit/7dcd4d13361df27516cf7ce5963bcf1e88ad90a7) Removed unused imports, +VALUE_TYPES import - [`8b56d64`](https://github.com/appy-one/acebase/commit/8b56d647657038a449d641f49b710d315d36ef6d) Added BigInt index query (+cache) unit tests - [`0dd7006`](https://github.com/appy-one/acebase/commit/0dd70063f4c2341a97d3ec1b99b7d05f7a0668c3) updated core dependency - [`1de236e`](https://github.com/appy-one/acebase/commit/1de236ee4d8dcc8f618355d348fe877b653f3533) chore: ESLint fixes - [`9b08026`](https://github.com/appy-one/acebase/commit/9b0802660b48a547e542b175bad9070b54f5431e) chore: build ### 📊 Changes **8 files changed** (+94 additions, -44 deletions) <details> <summary>View changed files</summary> 📝 `dist/browser.js` (+2 -2) 📝 `dist/browser.min.js` (+1 -1) 📝 `package.json` (+1 -1) 📝 `spec/indexes.spec.js` (+20 -20) 📝 `spec/query.spec.js` (+53 -1) 📝 `src/data-index/index.js` (+7 -8) 📝 `src/data-index/index.js.map` (+1 -1) 📝 `src/ts/data-index/index.ts` (+9 -10) </details> ### 📄 Description * Fixes `BigInt`s not being indexed * Fixes #141 (caching of queries with `BigInt`s in filters/take/skip) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>