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

initial-work1:Implement JSONRPC as a wire protocol #65

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gracebish
Copy link

Motivation:
Identify the right mapping between vertx messages and Json protocol given what the event bus does.
Explain here the context, and why you're making that change, what is the problem you're trying to solve.
@pmlopes
I have created an EventBusClient which will help to connect the event bus to the EventBusBridge. Kindly review and advise me accordingly.
Conformance:

Your commits should be signed and you should have signed the Eclipse Contributor Agreement as explained in https://github.com/eclipse/vert.x/blob/master/CONTRIBUTING.md
Please also make sure you adhere to the code style guidelines: https://github.com/vert-x3/wiki/wiki/Vert.x-code-style-guidelines

@gracebish
Copy link
Author

@pmlopes I am still working on this. I have pushed with some build failures though. I hope to rectify them on Sunday. I have created integrated tests by adding builds, dependencies, plugins and resources to the pom.xml file, adding a json file and a Rest integration test class. I hope to continue from where i have stopped. I however need more guidance on this.

@gracebish gracebish closed this Oct 22, 2021
@gracebish gracebish reopened this Oct 22, 2021
@gracebish
Copy link
Author

@pmlopes I am trying my best to understand the project requirements and as if i am lost in the middle of no where. I need more support on this. Could you kindly take me through the project requirements so that i can better understand?

@gracebish
Copy link
Author

Removed the Rest test class, the additional dependencies to the pom.xm and the json file

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

Successfully merging this pull request may close these issues.

1 participant