yumyv / epam_fe_2019_myronets

Epam study project on native js
Apache License 2.0
0 stars 0 forks source link

HW16-17-Ajax #16

Open Tttaras1 opened 4 years ago

Tttaras1 commented 4 years ago

Hello there.

• Why did you ignore eslint settings? image

• About stars: you should use one varriable to set post rating. for example post.countOfStars = 3.5; and based on this generate needed starts.

• Try to click on title with invalid data. image

• You should put this code to a seperate function (then and catch blocks). It will make your code more readable. image

• For what reason do you store your image path at your bakcend? image image

• The same thing: image

•All static data should be on front-end side!

• Why are you using different validation for equal inputs? image

• Where are the post that I just created?

image

• You have problem with one line text render: image

yumyv commented 4 years ago

Thank you for the feedback, I answered you in skype.