-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[WIP - Targeting Sep 2023] Add the messaging bridge pattern #4068
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@microsoft-github-policy-service agree |
|
Learn Build status updates of commit 51a2e4a: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
@MicrosoftDocs/cloud-architecture-content-team-pr-reviewers - Please review the proposed new article. If you want to move forward with the content, add a sign-off. #label:"aq-pr-triaged" |
|
#label:"cact-review-in-progress" |
|
@SzymonPobiega - thanks very much for your contribution. We'll be reviewing this to determine next steps, which might include pulling it into our private repo so we can do additional updates and position it correctly w/our patterns content. I will drop another comment and advise once we decide on next steps.. |
|
Learn Build status updates of commit abbff1e: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
Any updates or next steps on this one? |
|
Hi folks. Anything I can do to help this along for now? |
|
Hi @kbaley - apologies for the delay and thanks for checking back... it's just been a busy summer for us and we have a lot of folks out of office at various times. We have this on our August sprint for resolution though, so we will have some answers in a few weeks.. |
Co-authored-by: Udi Dahan <udidahan@users.noreply.github.com>
|
Learn Build status updates of commit 896a84f: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
Hey there @SzymonPobiega, we would like to get this included in our architecture patterns content catalog, so thank you very much for your investment of time. We will have some suggestions coming for you soon to help align this better with what we need. We won't be able to bring this in until September, but I'll be delivering you some feedback in the next business day or so on things you can do in Aug to help us get ready to go in September. So please watch this space for next steps! I'm moving this draft to PR status just to move it out of our normal process since it'll be open for another month or so. |
|
@SzymonPobiega & @kbaley, hoping to do an initial pass next week. I'll let you know! Thanks for all the effort I see you've put into it so far. @udidahan, have you gotten a chance to collab yet? |
udidahan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor wording improvements
Co-authored-by: Udi Dahan <udidahan@users.noreply.github.com>
|
Learn Build status updates of commit a04f0cc: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
ckittel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really good! Just one request on the image.
You can expect us to pick this up in September for publication. We'll leave it here/open for now, but you may see it get closed as it moves to one of our internal repos for final publication and editing.
cc: @BryanLa
|
Learn Build status updates of commit d33de9c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
udidahan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some more suggestions.
Co-authored-by: Udi Dahan <udidahan@users.noreply.github.com>
|
Thanks @udidahan, I applied those suggestions, with small tweaks to a couple. |
|
Learn Build status updates of commit ed21858: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
Learn Build status updates of commit 2423546: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
Learn Build status updates of commit e1bdf57: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
PRMerger Results
|
|
@SzymonPobiega, @kbaley, and @udidahan -- we are going to be closing this PR as we've opened on internally for publication (ref: MicrosoftDocs/architecture-center-pr#10955). It will still need to go through a review and editorial. We'll use this public PR (that I'm closing) for any necessary communication/updates along the way. Our goal is still to publish this month, and I don't see a reason why that shouldn't be able to happen. So, please watch this space for updates. #please-close |
|
@SzymonPobiega, @kbaley, and @udidahan -- the content is stagged and ready for editorial. I was told that editorial has this slated for ~3 weeks from now. They are backed up with some higher priority work at the moment. I'll provide updates here as I get them. cc: @mSarts (FYI only, no ask) |
|
One more update... Thank you to @SzymonPobiega, @kbaley, @udidahan, and @ckittel for your work on this contribution! The article will be published just as soon as the final URL is established. You'll also receive notification from our publishing team as soon as it's available. I'll also open a PR to the previously updated Migrate your e-commerce solution to Azure article, to include a link to this new pattern article. cc: @RobBagby |
|
It is wonderful seeing this come to fruition. Many thanks to everyone who helped get this done 🙏🏼 |
Before selecting the "Create pull request" button:
Enter a meaningful title above^, using a prefix if necessary and keywords "New" or "Update" indicating the nature of changes.
Describe the summary, scope, and intent of this PR:
This PR adds the messaging bridge pattern. The intent is to have this pattern listed in the category "messaging" in the pattern browser.
Based on https://www.enterpriseintegrationpatterns.com/patterns/messaging/MessagingBridge.html
AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors
Additional guidance for private repos and internal contributors
@MicrosoftDocs/cloud-architecture-content-team-pr-reviewersin your PR and someone will get back to you. After CACT review is complete, aCACT #sign-offwill be added.@MicrosoftDocs/cloud-architecture-content-team-pr-reviewers