Opened 13 years ago
Closed 10 years ago
#178 closed enhancement (fixed)
Have a Users page that lists all the users on a given install
Reported by: | Karen Rustad | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description (last modified by )
Right now, I have no way of telling how many users are on the site or any way of seeing their profiles (other than clicking on another user's photo and clicking their name on the photo page). If we're going to have user profiles, let's have a way of finding other users!
Change History (19)
comment:1 by , 13 years ago
comment:2 by , 13 years ago
Milestone: | → 0.2.2 |
---|---|
Owner: | set to |
Status: | accepted → assigned |
comment:3 by , 13 years ago
I made these changes, but I changed pagination.html so the paging widget now says "Next" and "Previous" instead of "Older" and "Newer"
Older and Newer just don't make sense for a list of users.
Also, this probably needs looking at by graphics/templating folks for two reasons:
- this page is only accessible by going to the /userlist.html page directly, there are no links to this list.
- It's a little ugly right now. Maybe a profile picture or something the user uploaded?
Changes are here:
https://gitorious.org/~chemhacker/mediagoblin/chemhackers-mediagoblin/commits/bug178_userlist
comment:4 by , 13 years ago
Owner: | changed from | to
---|
comment:5 by , 13 years ago
Description: | modified (diff) |
---|---|
Keywords: | needs-review added |
comment:7 by , 12 years ago
Keywords: | needs-review removed |
---|---|
Owner: | changed from | to
Type: | defect → enhancement |
Hi,
I looked at the patch.
It looks mostly good. I have to admit, I haven't tried it.
One improvement I would like to see before I merge this into master:
Move the check for status==active from the template into the db query. This should be as easy as ...db.User.find({status: 'active'})
(this is from my head, please verify it!). Just add a new commit to your branch changing this. And then assign the bug back to me and re-add a "needs-review" keyword/tag.
Thanks for your work!
comment:8 by , 12 years ago
Milestone: | 0.3.0 → 0.3.1 |
---|
comment:11 by , 12 years ago
Milestone: | 0.3.1 → 0.3.3 |
---|
No, this has never landed. I merged the conflicts in routing.py, and moved the check for active users into views.py. Some minor template cleanups thrown in and pushed the updated patch from chemhacker to the branch bug178_userlist at my repo at git://gitorious.org/~spaetz/mediagoblin/spaetz-mediagoblin.git
What remains to be done: Actually put a link to the user list on some template where it fits. Not sure where. Also the user list template is pretty bare bone still (but better than nothing).
comment:12 by , 12 years ago
You are right, I have no idea where to put a link to this list of all users.
comment:13 by , 12 years ago
I think the reason we can't think of a good spot for a link is that we can't think of a good reason why we should have that link. Wouldn't a search function be a better solution to Karen's problem?
comment:14 by , 12 years ago
You may be right. A user list might be interesting on smallish installs but it would be fairly useless on large installs.
comment:15 by , 12 years ago
Milestone: | 0.3.3 |
---|
At any rate I'm dropping this from 0.3.3 until we have a better idea of what to do.
comment:16 by , 12 years ago
Status: | assigned → in_progress |
---|
Hi! I'm moving this ticket from "assigned" to in_progress per our new workflow. Please update the ticket and let us know if you're still working on this. If you are, super great! If not, we'll remove the claim and move it back to "accepted" in a couple of weeks.
Thanks!
comment:17 by , 11 years ago
If we don't keep the user's list, then let us at the very least make an administrators dashboard that shows, the number of users, posted media etc. This could include a search field to search for individual users, the search results which could show up in a list, like the user's list requested in this ticket. This list could then also provide "deactivate" and "delete" buttons for users other than one self.
comment:18 by , 10 years ago
I believe this has been done, if you're an administrator you are able to go to /mod/users/
which will provide you with a user list. This was created as part of tilly-Q's moderation work. Specially the user list was introduced 6bba33 and refined to the current version in 6483b3.
Is this possible to be closed as fixed now?
comment:19 by , 10 years ago
Owner: | removed |
---|---|
Resolution: | → fixed |
Status: | in_progress → closed |
The original url for this bug was http://bugs.foocorp.net/issues/470 .