Skip to content

Discriminate against performAction tags for CallsFed constraints#3030

Merged
smatting merged 2 commits intowireapp:developfrom
isovector:discriminate-performAction
Jan 27, 2023
Merged

Discriminate against performAction tags for CallsFed constraints#3030
smatting merged 2 commits intowireapp:developfrom
isovector:discriminate-performAction

Conversation

@isovector
Copy link
Contributor

@isovector isovector commented Jan 26, 2023

This PR checks against the ConversationActionTag singleton in Galley's performAction in order to give finer-grained constraints about which federated calls get made.

Checklist

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

@isovector
Copy link
Contributor Author

@smatting please review

@smatting smatting self-requested a review January 27, 2023 08:53
@smatting smatting merged commit b5f1cb9 into wireapp:develop Jan 27, 2023
isovector added a commit to isovector/wire-server that referenced this pull request Feb 15, 2023
pcapriotti pushed a commit that referenced this pull request Feb 23, 2023
* feat: connect to transitive-anns

* feat: upgrade transitive-anns to ghc 9.2.5

* feat: first successful test!

* feat: upgrade to using ToHasAnnotations

* feat: remove CallsFed constraints

* chore: remove more CallsFed constraints

* chore: update hs pin

* doc: add documentation to new magical pieces

* feat: remove CallsFed constraints

* revert: #3030

* feat: try to make CallsFed more injective

* chore: re-remove CallsFed constraints

* chore: add plugin to cargohold

* fix: brig

* chore: use official transitive-anns pin

* doc: add changelog

* fix: get everything building

* chore: hlint

* chore: fix hlint

* fix nix

changes after running
```
./hack/bin/generate-local-nix-packages.sh
```

---------

Co-authored-by: Stefan Berthold <stefan.berthold@wire.com>
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.

2 participants

Comments