-
Notifications
You must be signed in to change notification settings - Fork 46
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
Enabling Functionality comments Section #5
Comments
@Pawankalyan2023 can you describe more about this issue? Thanks |
I think you have assigned someone with this task if you are okay with it i can contribute or else i will improve the contents |
@Pawankalyan2023 I'm assigning it to you, please look into it. Thank you for your contribution |
@Pawankalyan2023 i am working on the comments section and i am nearly done with it |
@NOOB-3301 did you made any PR against this issue ? |
nope i haven't made any pr till now, i did my changes locally, i have only written 2 models fro article and user , and controllers for addcomment, getarticle and addarticle . so i thoght you are also working on this controllers thats why i asked you |
But this issue is assigned to me right , then how you can start working on the part that I have already registered to work , Do you know how this assignment works ? @OkenHaha Please do check |
@Pawankalyan2023 you can work together with @NOOB-3301 as he suggested earlier. Each of you can make a different pull request on your own of which I'll have to edit the issue afterwards or you can combine it together and make a pull request. But I suggest what each of you will be working on after your discussion so that I can update the issue and you each can make a pull request of the problem you solve. Sounds good? |
@OkenHaha i made the pr, do check it out |
@NOOB-3301 @Pawankalyan2023 there's a bug in the code in which instead of making the api calls to the backend server it's calling it locally instead. Please change it to the server address (you can find the backend server link in navbar.jsx). I'll be updating the issue also so try checking the updated issue |
@OkenHaha Can you let me know if this issue is now resolved? |
@Pawankalyan2023 Ah yes, the issue is resolve now, I forgot to close this issue. Thank you for asking. |
Hey i will enable to comment section can you assign this issue to me under swoc 2025
Task to complete:-
The text was updated successfully, but these errors were encountered: