Skip to content
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

Challenge: don't show completed users as participants. #135

Closed
wants to merge 1 commit into from
Closed

Challenge: don't show completed users as participants. #135

wants to merge 1 commit into from

Conversation

phn
Copy link

@phn phn commented Jun 8, 2012

This PR address issue #127 .

Display page for a challenge shows help providers, participants, and people who have completed the challenge. The participant list included people who have completed the challenge.

We want to list only those who haven't completed the challenge in the participants list.

A simple "exclude" filter applied in project_tags to create a list "participants_not_completed". I inserted this in what I think are the appropriate locations in projects/_user_list.html.

I tested this using the

http://127.0.0.1:8000/en/groups/test-webmaking-101-7/

challenge that was created using the Django "loaddata" command, on the data in projects/fixtures.

See http://i.imgur.com/W1DSv.png .

Display page for a challenge shows help providers, participants,
and people who have completed the challenge. The participant list
included people who had completed the challenge.

We want to list only those who haven't completed the challenge
as participants.

A simple "exclude" filter applied in project_tags to create a
list "participants_not_completed". I inserted this in what I
think are the appropriate locations in projects/_user_list.html.

I tested this using the

http://127.0.0.1:8000/en/groups/test-webmaking-101-7/

challenge that was created using the Django "loaddata" command,
on the data in projects/fixtures.

See http://i.imgur.com/W1DSv.png .
@dirkcuys dirkcuys closed this Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants