search

foyzulkarim / mern-video-streaming

MERN Video Streaming is a cutting-edge, open-source platform for video streaming, offering a comprehensive, full-stack solution utilizing the latest MERN stack technologies.
MIT License
182 stars 63 forks source link

feat: increase video view count #23

Closed aninda052 closed 1 year ago

aninda052 commented 1 year ago

This pr will solve task #21.

aninda052 commented 1 year ago

@foyzulkarim vai, is it possible to give any feedback on this pr ?

foyzulkarim commented 1 year ago

@aninda052 I have checked the suggestions. Actually we will make a common base-repository later which will have the common update function. In that common update function we will send the payloads from the Service file. We should not add the simplification on the current service.js file. So, please add new functionas as I suggested.

Plus, please check I said earlier, we want to perform some other operations when we update (the operations may be different), and we MUST not do those operations on the controller.js file.

I hope you get the idea. Please let me know if you have other questions. Thanks.