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

feat: mqtt event handler #34

Merged
merged 31 commits into from
Aug 2, 2021

Conversation

NektariosFifes
Copy link
Collaborator

@NektariosFifes NektariosFifes commented Jul 23, 2021

Description

  • Able to send mqtt publish requests to channels specified based on eps (events per second) field in scenario file.
  • Added choice for targeted server on cli.
  • Payload specified in scenario file
  • Changes in readme
  • Cli refactoring. Cleaner code , fully asynchronous.
  • Added example scenario file for game-processor.

Related issue(s)
fixes #28

@NektariosFifes NektariosFifes mentioned this pull request Jul 25, 2021
@NektariosFifes NektariosFifes changed the title Mqtt event handler feat: Mqtt event handler Jul 26, 2021
@NektariosFifes NektariosFifes changed the title feat: Mqtt event handler feat: Mqtt event handler #28 Jul 26, 2021
@NektariosFifes NektariosFifes changed the title feat: Mqtt event handler #28 feat: Mqtt event handler Jul 26, 2021
@NektariosFifes NektariosFifes changed the title feat: Mqtt event handler feat: mqtt event handler Jul 26, 2021
@NektariosFifes NektariosFifes marked this pull request as ready for review July 27, 2021 11:30
Copy link
Sponsor Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

Got a bunch of change suggestions, most of them are small changes and nitpicks 🙂

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
example-projects/game-processor/config.js Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
@jonaslagoni
Copy link
Sponsor Member

@NektariosFifes can I make another review, or are you still missing a few things? 🤔

@NektariosFifes
Copy link
Collaborator Author

@NektariosFifes can I make another review, or are you still missing a few things? 🤔

Eveything is done

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/RequestHandler/RequestManager.js Outdated Show resolved Hide resolved
src/RequestHandler/mqttHandler.js Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
src/bin/cli.js Outdated Show resolved Hide resolved
NektariosFifes and others added 9 commits July 30, 2021 14:03
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Co-authored-by: Jonas Lagoni <jonas-lt@live.dk>
Copy link
Sponsor Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

Please remove the src/logger/index as there is no functionality 🙂

Also, please don't mark conversations as resolved if they are not 😄 Once you have committed the changes that addresses the comment or if another solution have been found, first then resolve the conversation 🙂

Copy link
Sponsor Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

Got a problem with the last commit 🙂

src/RequestHandler/RequestManager.js Outdated Show resolved Hide resolved
src/RequestHandler/RequestManager.js Outdated Show resolved Hide resolved
Copy link
Sponsor Member

@jonaslagoni jonaslagoni left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jonaslagoni jonaslagoni merged commit 7383189 into asyncapi:master Aug 2, 2021
@NektariosFifes
Copy link
Collaborator Author

ok now i will add testing

@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 0.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Event Handler MQTT
3 participants