Skip to content
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

Fixes/presales #91

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Fixes/presales #91

wants to merge 14 commits into from

Conversation

phofmann
Copy link
Collaborator

@phofmann phofmann commented Nov 4, 2024

Description and context

We are using the template in presales a lot to show how easy it is to build events using connect. What we need to do so is a rapid development roundtrip. We achieve this by writing tests using them to e.g. trigger a customer created message.

Type of change

  • New feature

Brief description of the solution

IDEs like idea allow you to run tests right from within the IDE, taking away the pain of doing things e.g. via postman. On top the message payload is also versioned within git. On top, a lot of clients will use the template to build other subscriptions than customerCreated, so I renamed the code references.

Copy link

changeset-bot bot commented Nov 4, 2024

⚠️ No Changeset found

Latest commit: e6efcae

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ruidias-commercetools ruidias-commercetools added question Further information is requested do-not-merge Please be so kind and do not merge it yet labels Nov 8, 2024
@ruidias-commercetools
Copy link
Contributor

Please do not merge yet. Thanks 🙏🏻

@phofmann
Copy link
Collaborator Author

@ruidias-commercetools is there anything I can do to move this PR forward?

@ruidias-commercetools
Copy link
Contributor

Hey @phofmann . We have @dario-commercetools working on this. Expect news from him on Monday. Thanks for the wait.

@phofmann
Copy link
Collaborator Author

Cool, thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge Please be so kind and do not merge it yet question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants