-
Notifications
You must be signed in to change notification settings - Fork 26
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
Implement feature where User can see how many Issues they have solved Level Wise #169
Comments
Hello @meisabhishekpatel , |
@VishalMinj for working on this issue you need to request through contrihub website |
i am not able to understand properly the code base like could you please help how should i do it it's a lot of .py file i am not able to comprehend i got that frontend thing but the functionality is all verbose to me |
@wenayy if you didn't knew anything about the tech stack why did you straight away jumped into solving the issue |
I can explain you some part of codebase which you are having any doubt with but if you don't know a tech stack then how can I explain you the whole tech stack |
the issue was simple but a lot of .py file overwhelmed me |
@wenayy i know that, but you should atleast know in what you are getting your hands into, |
Please assign me this issue @meisabhishekpatel |
@VishalMinj did you request through contrihub website ? |
Yes! |
hey @meisabhishekpatel what CSS lib. have you used in here? |
@VishalMinj well it's mostly bootstrap only you see all the cdn link here |
ok, I have one more doubt? are the tag info stored some where in the db or we need to get it through github? |
hey @meisabhishekpatel |
@VishalMinj i don't fully understand what you are trying to ask but but yes we don't a html page fully, rather that card is an component of user profile page you can see that in userprofile/templates folder, for accepting the PR we have a route which can be hit by mentor to accept the request and an email will be sent to the other user Also about the table creation it's fine to create a dedicated table for holding information but you, it would be much better if you will solve this issue without explicit creation because you will be storing redundant information. |
Thanks for responding @meisabhishekpatel , I can use that info, iterate over it and populate the card on the profile. |
you can see this https://github.com/ContriHUB/ContriHUB-24/blob/main/home/views.py for reference line number 275 |
I think I'm done now! My solution to filter issue from Pullrequest is working, for the most part. The email service was not working so I did manual transection and pr acceptance from admin. And it worked as soon as i accepted the pr status. If you will allow then i'd like to make pr. The only problem is, that i'm not assigned from contrihub website so how will i make the pr request over there? |
what do you mean by this.
why ?? |
What i mean is i haven't tried it via email acceptance. That's how it works right? By sending email to the auther and then he accepts or reviews. Well i tried my best to set up the email config. But it was throwing some error regarding thread. So I did the issue assignment and Pr acceptance from.the admin panel |
@VishalMinj what's the error exactly this might break things so you need to test that and then make PR |
Well, Here's the error. Occurs when I try to access any of mail functionality from helper.py Probably coming from somewhere around here As far as thing go on breaking I haven't touched much only added an objects query in the profile view and the card component. |
@VishalMinj did you properly setup the env ? |
Also show me complete error, also what are you filling in env to setup email service |
Was there a need to give an email id while creating a super user? |
no, you can use any temp mail for creating a super user |
@VishalMinj are you using this on college proxy or not ? |
No, i shifted to cellular while testing |
hey @meisabhishekpatel, I manage to receive the email Apparently this exception disappeared after I upgraded my django version. Now, how to accept the assignment request ? |
You have to go to maintainers userprofile to accept |
Ok I'll review it can you send PR through contriHUB website |
Done |
@VishalMinj solve for very-easy tag as well |
@VishalMinj create a new PR with requested changes and tag this issue as well in the PR |
@VishalMinj maintain consistency (very-easy Prob -> very-easy issue) |
@VishalMinj also create a PR request |
Feature Request: Display Level-Wise Issues Solved by User on User's Profile
Description:
We want to implement a feature where users can view the number of issues they have solved, categorized by difficulty level. This should be similar to how LeetCode displays problems as Easy, Medium, and Hard, with counts for each.
Acceptance Criteria:
UI Requirements:
The text was updated successfully, but these errors were encountered: