Skip to content

Implement Operation and EnvelopeBodyContents for &T#80

Merged
babolivier merged 3 commits intomainfrom
babolivier/operation_ref
Feb 13, 2026
Merged

Implement Operation and EnvelopeBodyContents for &T#80
babolivier merged 3 commits intomainfrom
babolivier/operation_ref

Conversation

@babolivier
Copy link
Member

@babolivier babolivier commented Feb 13, 2026

As part of the changes I'm working on for bug 2010330, I'd like to make it possible for an Envelope's body to be a reference on the operation struct instance rather than the instance itself. The reason is that I'm wrapping the instance into a struct that can be interacted with from the queue, and I'd like to be able to build the envelope without having to move the inner operation out of that struct.

Requires thunderbird/xml-struct-rs#11

@babolivier babolivier requested a review from a team as a code owner February 13, 2026 17:16
Copy link
Contributor

@jtracey jtracey left a comment

Choose a reason for hiding this comment

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

I'm a little nervous about using identical names to the owned types, but they really should be identical operations with these definitions, so it should be fine.

@babolivier
Copy link
Member Author

babolivier commented Feb 13, 2026

I'm a little nervous about using identical names to the owned types, but they really should be identical operations with these definitions, so it should be fine.

Yep it should be fine, and they need to translate to the same XML element names or the EWS requests wouldn't work, so there isn't really an alternative here.

@babolivier babolivier merged commit 3553c0b into main Feb 13, 2026
2 checks passed
@babolivier babolivier deleted the babolivier/operation_ref branch February 13, 2026 19:15
moz-v2v-gh pushed a commit to mozilla/releases-comm-central that referenced this pull request Feb 19, 2026
This patch also pulls changes from the following PRs:

* thunderbird/ews-rs#79
* thunderbird/ews-rs#80
* thunderbird/xml-struct-rs#11

Differential Revision: https://phabricator.services.mozilla.com/D283349

--HG--
extra : amend_source : 3ef8d00af3b4f770fed52c9b43438b33c2426c9e
github-actions bot pushed a commit to thunderbird/comm-unified-l10n that referenced this pull request Feb 19, 2026
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