Skip to content

[WPB-22549] connect users to apps.#4970

Draft
fisx wants to merge 10 commits intodevelopfrom
WPB-22549-connect-user-to-app
Draft

[WPB-22549] connect users to apps.#4970
fisx wants to merge 10 commits intodevelopfrom
WPB-22549-connect-user-to-app

Conversation

@fisx
Copy link
Contributor

@fisx fisx commented Jan 21, 2026

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

Checklist

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

@fisx fisx changed the title [WPB-22549] connect users to apps. WPB-22549: connect users to apps. Jan 21, 2026
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 21, 2026
@fisx fisx changed the title WPB-22549: connect users to apps. [WPB-22549] connect users to apps. Jan 21, 2026
@fisx fisx added not-ok-to-test Not approved for running tests in CI, this label is ignored if ok-to-test also exists on a PR and removed ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist labels Jan 21, 2026
@fisx fisx force-pushed the WPB-22549-connect-user-to-app branch 5 times, most recently from 6bcfe66 to 14b7ab8 Compare January 27, 2026 16:14
@fisx fisx force-pushed the WPB-22549-connect-user-to-app branch from 14b7ab8 to 9d436c0 Compare January 28, 2026 15:26
@battermann battermann force-pushed the WPB-22549-connect-user-to-app branch from e1925d0 to 89c8d44 Compare February 3, 2026 17:11
@battermann battermann removed the not-ok-to-test Not approved for running tests in CI, this label is ignored if ok-to-test also exists on a PR label Feb 4, 2026
Nothing -> case userTeam u of
Nothing -> pure Nothing
Just tid -> do
mApp <- AppStore.getApp (userId u) tid
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Shouldn't this happen in UserStore?

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants