-
Notifications
You must be signed in to change notification settings - Fork 652
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
Activity tab of course page should have an option to show only new editors #5527
Comments
@ragesoss I would like to work on this issue, can you please assign it to me? |
@thekavikumar we don't assign issues, but you are welcome to work on it! please post here if you have any questions. |
Sure @ragesoss I'll work on it and will let you know if I have any questions. |
@ragesoss I'm not able to run the application locally, I've done all the setup and how do I start the web app? |
@thekavikumar once you've done all the setup, the command |
Yeah @ragesoss sir, I did that but I'm getting this error |
It looks like your database either not running, or not configured with appropriate username/password. |
is it redis-server? |
No, the mysql server. |
Okay, so I've to install the mysql server in my wsl first and start the service then use the guard command, right? |
Yes. |
@ragesoss I inspected the articles.json and users.json of a course to see how exactly the data is stored. |
There is no open PR. Let me know if anyone of you is active on this one or else I'll take the task. @fillingtothemomo @thekavikumar |
@prathamVaidya I'm working on it. |
Hey @prathamVaidya I'm currently stuck on a bug and will open a pr soon. There are other open issues that you can take up. Thanks |
@ragesoss fetchRevisionsFromWiki method in |
I think the first step would be to add the registration date to the set of data about each user that is included with |
So I should add the register date of each user in the file where it is fetching revisions from and store new editors as well and then display them using revision handler ? And how exactly do I go about adding the registration date to each user in the file? |
Something like that, I think. |
@ragesoss , when I access the registered_at attribute of the user model in users json builder it shows registered date as null. |
You should be able to get all that data into your database by running |
@ragesoss, I’d like to work on this. Could you please update me on the current status of this issue? If it's still open, may I know where the Editor's Download is located? I couldn’t find it." |
This is still open. There shouldn't be any 'Download' link involved with it. It should use data that is already available on the frontend via user.json (which can be edited to include registration date for each user, if it's not already present). |
Sir, I am referring to new editors (using the same definition of 'new' as we use in the Editors data downloads). Based on the discussion, I believe using the registration_date is not ideal because it shows when the user created their account, not the date they joined the course. My idea is to add a field in the course data that tracks the date a user joined the course, rather than relying on their Wikipedia registration date. This way, we can track new editors. Additionally, since you mentioned 'new' in the Editors data downloads, could you kindly clarify where the Editors data downloads are located? I’d like check it and implement it accordingly |
I'm unclear about what the date a user joined the course is for. The definition of 'new' we want to use here is if the account was registered after the start date of the course. |
Current Behavior:
The Activity tab shows recent edits by course participants (pulled client-side directly from wiki APIs). However, this feed is often dominated by the activity of active editors, if any, and can't easily be used to check for newcomers who are still active.
Desired Behavior:
Add a button to limit the Activity feed to only new editors (using the same definition of 'new' as we use in the Editors data downloads).
The text was updated successfully, but these errors were encountered: