[FEATURE REQ] Please add an option to set the application properties inside the ServiceBusModelFactory class about azure-sdk-for-net HOT 8 CLOSED

Renaming the parameters for the model factory method is a breaking change, which this doesn't meet the bar for. I've added the missing doc comments for the parameters, highlighting which property of the associated model they populate as part of #43276. This will be available as part of our April release this week.

from azure-sdk-for-net.

Thank you for your feedback. Tagging and routing to the team member best able to assist.

from azure-sdk-for-net.

Hi @jhueppauff. Thanks for reaching out and we regret that you're experiencing difficulties. Can you help us understand what model you're looking to set application properties for? The one that seems applicable to me is ServiceBusReceivedMessage where the ApplicationProperties dictionary can be passed using the properties parameter. (I don't know why the name of the parameter differs from the name of the property it matches, but I'll look into getting that updated, likewise with the missing parameter documentation.)

from azure-sdk-for-net.

Hi @jhueppauff. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

from azure-sdk-for-net.

Hi @jhueppauff, we're sending this friendly reminder because we haven't heard back from you in 7 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!

from azure-sdk-for-net.

sorry for the late response, yes you are right. Properties = ApplicationProperties.

The naming is indeed a bit confusing as it is kind of different at every place. Looking at Azure it is not even consistent there:

But from the SDK perspective I think it would really help to have it named application properties. As I kind was assuming that properties are the system properties like CorrelationId, etc

Thank you for your help :)

from azure-sdk-for-net.

Hi @jhueppauff. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text "/unresolve" to remove the "issue-addressed" label and continue the conversation.

from azure-sdk-for-net.

Hi @jhueppauff, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.

from azure-sdk-for-net.