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

Peer review - Milestone 4 - Summary #190

Open
uteerna opened this issue Apr 25, 2019 · 1 comment
Open

Peer review - Milestone 4 - Summary #190

uteerna opened this issue Apr 25, 2019 · 1 comment
Assignees

Comments

@uteerna
Copy link

uteerna commented Apr 25, 2019

Hi Team,
I liked the UI. It is simple and easy to navigate. I liked the implementation of the distributed micro-service architecture. However, I found the following issues during the review:

  • Sign up has a bug and it does not show anything after entering the details. Sign up issue #186
  • Length of song is 30 seconds. I am not sure if this is how it should play. Length of song played #188
  • The music player bar is located at the bottom of the page. It would be good if it was fixed so the user does not have to scroll all the way down. UI enhancement  #187
  • It would also be good if you have a previous or next button to navigate the songs. UI enhancement  #187
@sairohithA007
Copy link
Contributor

Hi Uteerna,

Thanks for the feedback. We will try to address each of them one by one.

Regards,
Sai Rohith Achanta.

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

No branches or pull requests

5 participants