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

feat: CoreOwnership class w getOwner & getCoreKey [2/3] #229

Merged
merged 3 commits into from
Aug 30, 2023

Conversation

gmaclennan
Copy link
Member

Stacked on #214

  • also writeOwnership

Required the addition of an internal select() method for dataType,
which directly accesses drizzle.select().from(table)

@gmaclennan gmaclennan self-assigned this Aug 28, 2023
@gmaclennan gmaclennan changed the title feat: CoreOwnership class w getOwner & getCoreKey feat: CoreOwnership class w getOwner & getCoreKey [2/3] Aug 28, 2023
@gmaclennan gmaclennan linked an issue Aug 28, 2023 that may be closed by this pull request
@gmaclennan gmaclennan linked an issue Aug 28, 2023 that may be closed by this pull request
7 tasks
src/core-ownership.js Outdated Show resolved Hide resolved
src/core-ownership.js Show resolved Hide resolved
@gmaclennan gmaclennan force-pushed the feat/core-ownership-index-writer branch 2 times, most recently from c4688cf to d815ca4 Compare August 30, 2023 14:13
Base automatically changed from feat/core-ownership-index-writer to main August 30, 2023 14:17
- also writeOwnership

Required the addition of an internal `select()` method for dataType,
which directly accesses drizzle.select().from(table)

Cleanup & fix types
@gmaclennan gmaclennan merged commit 7faa1d5 into main Aug 30, 2023
6 checks passed
@gmaclennan gmaclennan deleted the feat/core-ownership-class branch August 30, 2023 14:38
gmaclennan added a commit that referenced this pull request Sep 6, 2023
* main: (25 commits)
  add initial implementation of MemberApi (#232)
  feat: $blobs.getUrl and $blobs.create methods (#184)
  chore: update manager e2e tests (#237)
  feat: add capabilities (#231)
  feat: coreOwnership integration [3/3] (#230)
  feat: CoreOwnership class w getOwner & getCoreKey [2/3] (#229)
  feat: handle `coreOwnership` records in `IndexWriter` [1/3] (#214)
  fix: adjust storage options for MapeoManager and MapeoProject (#235)
  implement addProject method for MapeoManager class (#215)
  implement listProjects method for MapeoManager class (#208)
  feat: expose blobStore.writerDriveId (#219)
  implement wrapper client containing createProject and getProject methods (#199)
  add project settings functionality to MapeoProject (#187)
  feat: Add encode/decode for project keys [3/3] (#203)
  feat: update protobuf for RPC [2/3] (#202)
  chore: move protobuf messages into src/generated [1/3] (#201)
  feat: Add internal `dataType.createWithDocId()` (#192)
  chore: explicitly set "mode" opt for encryptionKeys column creation (#186)
  chore: fix linting and type checking (#183)
  chore: consolidate encryption key columns in projectKeys table (#181)
  ...
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.

Add core ownership records
2 participants