Added lecture to specific playlist

[Anushkabh]

Have you read the Contributing Guidelines?
yes

Issue #41

Describe your changes

Created a controller for adding lectures to playlist.
It first validates the existence of the specified course by its ID.
If the course exists, it checks if the lecture with the provided title already exists in the course. If so, it returns an error.
If the lecture is unique, it adds the lecture details (title, description, video public ID, and URL) to the course's lectures array.

Clearly and concisely describe what's in this pull request. Include screenshots, if necessary.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
edumi	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 18, 2024 3:00pm

[anand-harsh]

take just title and description from req.body and id from req.params

[anand-harsh]

Have you read the Contributing Guidelines? yes

Issue #41

Describe your changes

Created a controller for adding lectures to playlist. It first validates the existence of the specified course by its ID. If the course exists, it checks if the lecture with the provided title already exists in the course. If so, it returns an error. If the lecture is unique, it adds the lecture details (title, description, video public ID, and URL) to the course's lectures array.

Clearly and concisely describe what's in this pull request. Include screenshots, if necessary.

Hi, I am sorry I had a confusion in route which I mentioned in issue

Take the route as /course/:id

Also add isAuthenticated and adminAuthorise middlewares, so that only admin has power to add lectures in course

[Anushkabh]

@anand-harsh i have made the changes .. kindly have a look at it

[anand-harsh]

Hi @Anushkabh , have you tested this api on your local?

[anand-harsh]

Hi @Anushkabh , have you tested this api for this endpoint on your local?

[Anushkabh]

yes , but without using admin middleware

[anand-harsh]

yes , but without using admin middleware

send me screenshot of the request and response

[anand-harsh]

Go to models > Courses.js, comment out the required:true in lectures > video >public_id and url

[anand-harsh]

Hi @Anushkabh , your code works fine for all cases and admin auth too, just make the suggested changes and will be ready to merge

[anand-harsh]

LGTM💥💥

[Anushkabh]