Skip to content

Conversation

@juliapampus
Copy link
Contributor

@juliapampus juliapampus commented Jun 26, 2025

What this PR changes/adds

Adds editorial changes:

If the message initiates a [=Contract Negotiation=], it _MUST_ contain a `callbackAddress` property indicating where messages to the [=Provider=] _SHOULD_ be sent in asynchronous settings. If the address is not understood, the [=Consumer=] _MUST_ return an **unrecoverable** error.

Why it does that

Consistency in writing

Linked Issue(s)

Closes #13
Closes #114

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@juliapampus juliapampus force-pushed the refactor/editorial-changes branch from 8d9b48f to 4690a70 Compare June 27, 2025 08:35
@juliapampus juliapampus force-pushed the refactor/editorial-changes branch from 4690a70 to f07aa55 Compare June 27, 2025 08:45
@juliapampus juliapampus marked this pull request as ready for review June 27, 2025 08:46
@juliapampus juliapampus changed the title refactor: make editorial changes refactor: apply editorial changes Jun 27, 2025
@juliapampus juliapampus force-pushed the refactor/editorial-changes branch from f07aa55 to 7d5bf64 Compare June 27, 2025 09:38
@juliapampus juliapampus mentioned this pull request Jun 27, 2025
Copy link
Contributor

@arnoweiss arnoweiss left a comment

Choose a reason for hiding this comment

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

There's some change in normativity implied in these changes.

@juliapampus juliapampus force-pushed the refactor/editorial-changes branch from 4ed72e4 to 3bbe01c Compare June 27, 2025 12:09
@juliapampus juliapampus force-pushed the refactor/editorial-changes branch from 3bbe01c to b35081b Compare June 27, 2025 12:10
Copy link
Contributor

@arnoweiss arnoweiss left a comment

Choose a reason for hiding this comment

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

Thanks for incorporating the feedback

@juliapampus juliapampus merged commit f441cc3 into eclipse-dataspace-protocol-base:main Jun 27, 2025
3 checks passed
@juliapampus juliapampus deleted the refactor/editorial-changes branch June 27, 2025 12:47
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.

Introduce abbreviations Change normativity keywords to ISO conventions

3 participants