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

adding server functionality, example project and some code improvements #5

Merged
merged 1 commit into from
May 9, 2018

Conversation

BlackReloaded
Copy link
Contributor

Hi,

i added server functionality for #3, also i added a example project to show how it works

@BlackReloaded
Copy link
Contributor Author

i have just seen that my commit also fixed #4

@fajran
Copy link
Owner

fajran commented Mar 31, 2018

Hi, sorry for the late response. Thank you for your contribution! I will review it.

@aneshas
Copy link

aneshas commented May 8, 2018

Hey guys, do you have plans to merge this any time soon? Thanks.

@fajran fajran merged commit c71f16b into fajran:master May 9, 2018
@fajran
Copy link
Owner

fajran commented May 9, 2018

Sorry I have not really had time to properly review this code. But from what I see in the changes, it looks good. My only concern is about the ProtoBufMessageProvider class. I imagine there is a name clash possibility, but let's deal it later.

I will merge this change. Sorry for taking so long.

also, thanks for the contribution!

@aneshas
Copy link

aneshas commented May 9, 2018

@fajran thank you

tobiaskohlbau added a commit to tobiaskohlbau/twirp that referenced this pull request Jun 28, 2018
KarenEtheridg pushed a commit to KarenEtheridg/twirp that referenced this pull request Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants