[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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.

Sign up for GitHub

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

Jump to bottom

Teams writes switch #4883

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Teams writes switch #4883

wants to merge 46 commits into from

Conversation

zoldar
Copy link
Contributor
@zoldar zoldar commented Dec 5, 2024

Changes

Please describe the changes made in the pull request here.

Below you'll find a checklist. For each item on the list, check one option and delete the other.

Tests

  • Automated tests have been added
  • This PR does not require tests

Changelog

  • Entry has been added to changelog
  • This PR does not make a user-facing change

Documentation

  • Docs have been updated
  • This change does not need a documentation update

Dark mode

  • The UI has been tested both in dark and light mode
  • This PR does not change the UI

@aerosol aerosol force-pushed the teams-writes-switch branch from a313eae to 089a1a6 Compare December 9, 2024 15:49
zoldar and others added 23 commits December 10, 2024 14:19
@zoldar @aerosol
Comment out legacy fields and relationships
6cf6314
@zoldar @aerosol
WIP
f11d709
@zoldar @aerosol
WIP 2
cb1ae4d
@zoldar @aerosol
WIP 3
7d48e0a
@aerosol
wip
f9116b8
@zoldar @aerosol
Remove teams backfill and consistency check scripts
f8fae57
@zoldar @aerosol
WIP 3
9510b4c
@aerosol
Fix CheckUsage tests
9e53b9c
@aerosol
Update billing/subscription tests
a57d0ee
@zoldar @aerosol
WIP 4
feb0e0e
@aerosol
Make site transfer fail if some invitation already exists
d0ef203
@aerosol
Fixup: do symmetric invitation/site transfer check
4573939
@aerosol
Update UI bugs: make listing sites/inviting admins work like before
faa546c
@aerosol
Fix Sites test
45711ac
@aerosol
Fix external sites controller test
90fb543
@aerosol
Fix live sites tests
965395e
@aerosol
Fix props availability lookup
317e576
@aerosol
Fix site controller tests
f3f0874
@aerosol
Fix billing controller tests
e687623
@aerosol
WIP - accept invitation tests
491dfa1
@zoldar @aerosol
Another round of test fixes + invitations logic bugs
8e35f8f
@aerosol
users_test -> teams_test
a2fb761
@aerosol
Update registration via invitation
148ae9c 
Here, we still rely on "polymorphic" invitation structures,
hence the "unified by id" helper.

For now, it'll remain local unless we discover it's
needed in the broader `Teams.Invitations` context.

cc @zoldar
@aerosol aerosol force-pushed the teams-writes-switch branch from 02cc31e to 148ae9c Compare December 10, 2024 13:19
@zoldar zoldar force-pushed the teams-writes-switch branch from bfaa48d to 1267eb2 Compare December 10, 2024 20:07
@zoldar zoldar added the preview label Dec 11, 2024
@github-actions GitHub Actions
Copy link
Preview environment👷🏼‍♀️🏗️
PR-4883

@aerosol aerosol marked this pull request as ready for review December 11, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aerosol aerosol aerosol approved these changes

Assignees
No one assigned
Labels
preview
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zoldar @aerosol