Skip to content

Conversation

Gaelik-git
Copy link
Contributor

Following this documentation : https://learn.microsoft.com/en-us/rest/api/servicebus/message-headers-and-properties#message-headers

I would like the possibility to set an ScheduledEnqueueTimeUtc (other option could be added in the future)

I updated the existing send_message method, making to modification a breaking change. Do you think it's ok like this ?
Should I create a new method for sending messages with options set ?

Thanks

Copy link
Member

@heaths heaths 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 improving Service Bus SDK! One blocking comment, though. I'm okay with the break in this case, but let's not special-case too much when we do that. With an options bag, we can add more in the future without breaking.

Copy link
Member

@heaths heaths left a comment

Choose a reason for hiding this comment

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

Thank you! I just have one question/nit but otherwise LGTM. Once I get this merged, I'll publish new versions.

@Gaelik-git
Copy link
Contributor Author

Everything should be ready, thanks for the feedback

@heaths
Copy link
Member

heaths commented Aug 12, 2024

Thank you for making the requested changes!

@heaths heaths enabled auto-merge (squash) August 12, 2024 21:58
@heaths heaths merged commit 6409e82 into Azure:main Aug 12, 2024
@Gaelik-git Gaelik-git deleted the adding_brokerproperties_when_sending_message branch August 14, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants