Skip to content

Conversation

@azarboon
Copy link

@azarboon azarboon commented Oct 7, 2024

Describe your changes

This page should (at least briefly) mention the main challenges of EDA so readers are informed. I've added two main challenges.

Related issue number or link (ex: resolves #issue-number)

Checklist before opening this PR (put x in the checkboxes)

  • This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@netlify
Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit b00ded8
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6703692f7700f600083f2985
😎 Deploy Preview https://deploy-preview-3318--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@iamNoah1 iamNoah1 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 the contribution @azarboon. I am not sure if this adds value for readers who want to get a brief understanding what event driven architecture is all about.

What you think @seokho-son @jihoon-seo @nate-double-u

@nate-double-u nate-double-u changed the title Update event-driven-architecture.md [EN] update event-driven-architecture Apr 7, 2025
@nate-double-u
Copy link
Member

Hi @azarboon, sorry for the delay in review.

In addition to @iamNoah1's comments above, we can't accept this update as it breaks our definition template.

@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in CNCF glossary definition review board Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/en for English

Projects

Development

Successfully merging this pull request may close these issues.

3 participants