Skip to content

[WPB-8887] wire-subsystems: implement the GetBy* account queries, includes InvitationCodeStore.#4218

Merged
fisx merged 102 commits intodevelopfrom
mangoiv/wpb-8887
Sep 12, 2024
Merged

[WPB-8887] wire-subsystems: implement the GetBy* account queries, includes InvitationCodeStore.#4218
fisx merged 102 commits intodevelopfrom
mangoiv/wpb-8887

Conversation

@MangoIV
Copy link
Contributor

@MangoIV MangoIV commented Aug 19, 2024

https://wearezeta.atlassian.net/browse/WPB-8887

  • new Effect operation GetAccountBy in UserSubsystem
  • new record GetBy
  • new stores ActivationCodeStore and InvitationCodeStore
  • new sql quasiquoter in cassandra-util
  • some more Ord instances derived
  • new function tSplit for the use with ViewPatterns
  • drive-by cleaning up invitation accessors

The remaining part of the ticket will be addressed in a separate PR.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@echoes-hq echoes-hq bot added the echoes: technical-roadmap/throughput More specific category, to highlight task aiming at improving the development velocity and effici... label Aug 19, 2024
@MangoIV MangoIV changed the title [WPB-8887] initial impl for GetBy* account queries as Effect and interpreter [WPB-8887] wire-subsystems: implement the GetBy* account queries Aug 19, 2024
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Aug 19, 2024
@mdimjasevic mdimjasevic self-requested a review August 21, 2024 09:58
@elland elland force-pushed the mangoiv/wpb-8887 branch 3 times, most recently from 13c7eaf to 6fbb7ec Compare August 27, 2024 14:23
@fisx fisx force-pushed the mangoiv/wpb-8887 branch from 7562f16 to 6fbb7ec Compare August 29, 2024 08:06
@elland elland marked this pull request as ready for review August 29, 2024 08:06
@fisx
Copy link
Contributor

fisx commented Sep 2, 2024

didn't we decide to leave account stuff out of this PR, and make it only about Invitations?

Copy link
Contributor

@elland elland left a comment

Choose a reason for hiding this comment

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

image

@fisx
Copy link
Contributor

fisx commented Sep 12, 2024

this took wayyyy too long, but not wayyyyyyyy to long :) i think the result is quite valuable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: technical-roadmap/throughput More specific category, to highlight task aiming at improving the development velocity and effici... ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants