Skip to content

Conversation

DanielePalaia
Copy link
Contributor

@DanielePalaia DanielePalaia commented Feb 13, 2025

This closes #35

Gives the possibility to Publish with addresses specified directly within the message (and not in the Publisher instance)

It also add some test refactoring

@DanielePalaia DanielePalaia marked this pull request as draft February 13, 2025 10:05
@DanielePalaia DanielePalaia force-pushed the publishing_per_message_target branch 28 times, most recently from a94efc8 to c25c1f3 Compare February 14, 2025 09:19
@DanielePalaia DanielePalaia force-pushed the publishing_per_message_target branch 11 times, most recently from baa198e to 86637bc Compare February 14, 2025 10:52
@DanielePalaia DanielePalaia force-pushed the publishing_per_message_target branch from 86637bc to 2744c14 Compare February 14, 2025 13:11
@DanielePalaia DanielePalaia changed the title [DRAFT] making Message a client class [DRAFT] publish per-message address Feb 14, 2025
@DanielePalaia DanielePalaia force-pushed the publishing_per_message_target branch from 4680ac9 to fa4470f Compare February 14, 2025 13:39
@DanielePalaia DanielePalaia force-pushed the publishing_per_message_target branch from fa4470f to ce91405 Compare February 14, 2025 13:40
@DanielePalaia DanielePalaia changed the title [DRAFT] publish per-message address Publish per-message address Feb 14, 2025
@DanielePalaia DanielePalaia marked this pull request as ready for review February 14, 2025 14:22
@DanielePalaia
Copy link
Contributor Author

Hey @Gsantomaggio I finally removed the AMQPMessage layer in place of an Utility Function as discussed.
What do you think?

Copy link
Member

@Gsantomaggio Gsantomaggio left a comment

Choose a reason for hiding this comment

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

Can you address the comments?

@DanielePalaia
Copy link
Contributor Author

@Gsantomaggio Done!

@DanielePalaia DanielePalaia merged commit 22c8f32 into main Feb 14, 2025
1 check passed
@DanielePalaia DanielePalaia deleted the publishing_per_message_target branch February 14, 2025 15:35
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.

Implement publishing with "per message" base target
2 participants