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

Create a server program to accept session feedback request #68

Closed
hkurokawa opened this issue Feb 5, 2016 · 9 comments
Closed

Create a server program to accept session feedback request #68

hkurokawa opened this issue Feb 5, 2016 · 9 comments
Assignees
Milestone

Comments

@hkurokawa
Copy link
Collaborator

This is a dependent of #32 Add feedback function.

Create a server program which accepts a session feedback request like below:

POST https://droidkaigi.jp/feedback

{
  "sessionId": 1,
  "ratingValuability": 2,
  "ratingContents": 3,
  "ratingSpeaker": 3,
  "comment": "Good. But not best IMO."
}

Requirements

  • Can process ~50 req/sec
  • Average response time is less than 200 msec
  • Should be working for 4 days, from 18th Feb to 21th Feb
  • Data should be stored so that it can be summarised for each session later.
  • The size of comment is less than 1.2KB (400 characters in Japanese)
@hkurokawa hkurokawa added this to the 1.0.0 milestone Feb 5, 2016
@konifar
Copy link
Owner

konifar commented Feb 5, 2016

👀

@konifar
Copy link
Owner

konifar commented Feb 5, 2016

Should comment have max langth?

@konifar
Copy link
Owner

konifar commented Feb 5, 2016

The api looks good for me 😃

@hkurokawa
Copy link
Collaborator Author

Great point! Added that limitation.

@konifar
Copy link
Owner

konifar commented Feb 5, 2016

Thanks!
And if you use RDB, I think sessionId should have INDEX.

@hotchemi
Copy link
Contributor

hotchemi commented Feb 6, 2016

@hkurokawa @konifar

Just created sample. Please check readme. 3 minutes cooking:D

https://github.com/hotchemi/feedback-api

@konifar
Copy link
Owner

konifar commented Feb 6, 2016

👀
Thanks! I'll check it later.

@konifar
Copy link
Owner

konifar commented Feb 7, 2016

@hkurokawa hkurokawa self-assigned this Feb 8, 2016
@konifar
Copy link
Owner

konifar commented Feb 10, 2016

Close this issue because we use google apps

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

No branches or pull requests

3 participants