Skip to content

[WebPubSub] Prepare for july 2021 release#22615

Merged
Mohit-Chakraborty merged 11 commits intomainfrom
mohitc/WebPubSubJulyRelease
Jul 15, 2021
Merged

[WebPubSub] Prepare for july 2021 release#22615
Mohit-Chakraborty merged 11 commits intomainfrom
mohitc/WebPubSubJulyRelease

Conversation

@Mohit-Chakraborty
Copy link
Copy Markdown
Contributor

No description provided.

@Mohit-Chakraborty Mohit-Chakraborty marked this pull request as ready for review July 13, 2021 20:18
Comment thread sdk/webpubsub/Azure.Messaging.WebPubSub/CHANGELOG.md
Copy link
Copy Markdown
Member

@tg-msft tg-msft left a comment

Choose a reason for hiding this comment

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

Looks good to me

Comment thread sdk/webpubsub/Azure.Messaging.WebPubSub/src/properties/AssemblyInfo.cs Outdated
Comment thread sdk/webpubsub/Azure.Messaging.WebPubSub/CHANGELOG.md Outdated
Comment thread sdk/webpubsub/Azure.Messaging.WebPubSub/CHANGELOG.md Outdated
Comment thread sdk/webpubsub/Azure.Messaging.WebPubSub/CHANGELOG.md
- Added support for API management.
- Simplified dependencies.

### Bugs Fixed
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @KrzysztofCwalina, is Azure/azure-webpubsub#47 fixed in this release?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The new swagger has minLength 1 paths, so it's theoretically fixed. It would be good to validate the fix too, but the repro steps have a broken link to repro code.

Mohit-Chakraborty and others added 2 commits July 14, 2021 09:16
Co-authored-by: Liangying.Wei <lianwei@microsoft.com>
@Mohit-Chakraborty Mohit-Chakraborty merged commit ce9cc9a into main Jul 15, 2021
@Mohit-Chakraborty Mohit-Chakraborty deleted the mohitc/WebPubSubJulyRelease branch July 15, 2021 01:12
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.

5 participants