ContriHUB / Tech-Mirror

First individual project
0 stars 14 forks source link

Make routes #10

Closed the-evocative-eye closed 1 year ago

the-evocative-eye commented 1 year ago

We have a chat icon in the header.ejs, but does not have its implementation as of now. Therefore, try making routes for it, for which we are supposed to implement a chat feature on our web(it must be three to four lines work at max).

For more consideration go to the routes folder, which consists of two basic routers for POST requests (search and display all), similarly, you might have to make a new file within it to move further and then configure that file in our main file(app.js).

You can refer to this guide to study more about routing https://expressjs.com/en/guide/routing.html

Sahilgupta3224 commented 1 year ago

do we need to implement this by socket?

the-evocative-eye commented 1 year ago

You don't have to develop a chat system, just form routes for the chat icon in the navbar so that when we click over, it redirects us to some other http.

Sahilgupta3224 commented 1 year ago

ok i can do it please accept my request

rishuxd commented 1 year ago

Hey @the-evocative-eye, I'd like to work on it, will try to drop the PR by tomorrow.

rishuxd commented 1 year ago

@the-evocative-eye Fortunately, I did it by now, I've submitted a PR, have a look at it.

Sahilgupta3224 commented 1 year ago

please assign me this issue, i have sent the request.

the-evocative-eye commented 1 year ago

It is already assigned to somebody, if that doesn't work out, the issue might remain open, then you can work on it!

On Wed, 11 Oct, 2023, 1:09 pm Sahilgupta3224, @.***> wrote:

please assign me this issue, i have sent the request.

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/Tech-Mirror/issues/10#issuecomment-1757030585, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXHEHI2RU33UMOIKUFC6SDLX6ZEKXANCNFSM6AAAAAA52N3GXU . You are receiving this because you were mentioned.Message ID: @.***>