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

sponsorship: admin benefit-sponsor relation #84

Merged
merged 3 commits into from
Sep 8, 2015

Conversation

miurahr
Copy link
Contributor

@miurahr miurahr commented Jun 21, 2015

Feedback from PyCon development
Here picks sponsorship benefit management enhancement

commit eb3261c12c910ec562e016f10431cc48747baef8
Author: Dan Poirier <dpoirier@caktusgroup.com>
Date:   Wed Aug 21 11:51:20 2013 -0400

    Enhanced sponsor admin page

    For #67:

    * admin list sorted by name
    * not limited to 100 per page
    * a sortable visual indicator for each sponsorship benefit
      (completed, missing, not applicable)
      also the sponsorship level, admin contact name, and an active or
      not indication
    * the  ones we have today: print logo, web logo, print description,
      web description and the ad.
    * an action pick list like “email” and check mark the sponsors I
      want to email based on the assets that are missing.
      in subject and body, replace %%NAME%% by sponsor name

Signed-off-by: Hiroshi Miura miurahr@linux.com

@miurahr miurahr force-pushed the admin_sponsor_benefit branch from 0a637a3 to 464f08f Compare June 21, 2015 06:27
miurahr added 3 commits July 11, 2015 11:27
Signed-off-by: Hiroshi Miura <miurahr@linux.com>
Signed-off-by: Hiroshi Miura <miurahr@linux.com>
Signed-off-by: Hiroshi Miura <miurahr@linux.com>
@miurahr miurahr force-pushed the admin_sponsor_benefit branch from 989d71b to d174008 Compare July 11, 2015 02:31
@miurahr
Copy link
Contributor Author

miurahr commented Jul 11, 2015

Update PR; each commit has its clear modification.

@paltman paltman merged commit d174008 into pinax:master Sep 8, 2015
@miurahr miurahr deleted the admin_sponsor_benefit branch September 14, 2015 13:38
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