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.

Sign up for GitHub

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

Jump to bottom

Demi project owner list of pledgers #36

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Demi project owner list of pledgers #36

wants to merge 64 commits into from

Conversation

demileee
Copy link

No description provided.

achristens and others added 30 commits October 26, 2017 14:35
@achristens
Dont allow project to save without user
4750073
@achristens
Don't allow pledge user to be project owner
9d7b930
@achristens
Add validations for reward model
f89abf5
@achristens
Update pledges controler
be6e7e1
Create validation to make goal positive
6ab9757
Write test to check that project goal must be greater than 0
1c74c53
Write test to check that project goal must be a positive number
4eaf54a
Create validation for dollar amount to be positive
226db31
Create test for reward dollar to be 0
404d531
Create test for reward dollar to be a positive value
676f7cf
@joshdales
Merge pull request #1 from joshdales/josh-and-demi-validations
cbb04b2 
Josh and demi validations
@cece24
Add pledge test for presence of dollar amount; Add project validation…
f87b776 
… for date not in the past, add project test for this validation
Display total dollar amount raised for each project
184ed9f
@cece24
Add project validation for project end date is after start date, add …
37b26ef 
…corresponding test
Create helper method for displaying dollar amounts to 2dp
43268c2
@cece24
Merge branch 'master' into cece-oct-26
a690af7
@cece24
Merge pull request #2 from joshdales/cece-oct-26
fa691a0 
Cece oct 26
Put time remaining on project show page
c973e94
Add owners update link to project show page
cd77ec9
@achristens
Working on date:nil for new projects
6305914
@achristens
Merge pull request #3 from joshdales/abby-oct-26
6a48ca3 
Working on date:nil for new projects
Create update table and test that it saves
b2e0474
@cece24
Resolve project start and end date issue, will now create a date inst…
441e67d 
…ead of nil
@cece24
Merge pull request #4 from joshdales/cece-oct-26-2
53fcc6d 
Resolve project start and end date issue, will now create a date inst…
Rename updates table to project_updates
e4feb7a
Merge branch 'master' of github.com:joshdales/crowdfunder
e7f72f2
Create validations for ProjectUpdate
ec88d35
Create test to check validation of project_id in ProjectUpdate
bfb5327
Create test for ProjectUpdate.title valid?
1c1215d
Create test for ProjectUpdate.description valid?
7abfa2f
cece24 and others added 30 commits October 27, 2017 00:31
@cece24
Save optional limit attribute to Reward instance in create action, ad…
7910bf2 
…d form field for limit in new reward view
@cece24
Add text displaying reward limit on project show view
0e9c7e4
@cece24
Add claimed attribute to Rewards to track how many instances of each …
6f5c837 
…rewards has been claimed
@cece24
Add project rewards update route, add reward update action to increas…
7a43766 
…e claimed count by 1 if claimed, add button to claim reward and display claim count in project show view
@cece24
Add new patch route for claiming a reward, add new action named claim
5e073b2
@cece24
Change formatting of no claim limit text on project show view
2594682
Add linkedin logo to image assets
2cde33f
Change negative test values to -1
283ad66
@cece24
Add reward_id column to pledges table
b9d2e8a
@cece24
Add 1 to many association between Reward and Pledge
a79f6f8
Create views for product_update and views for new, edit, and show
f5897c0
Create partial view for product_update form and set variables for new…
aeea0f8 
… and edit in the contoller
Fix links to New Update page and fix typos in the forms
175f321
Create links from the update forms back to the project pages
5a2cabd
Set parmas to save project_update
f61b74a
Change dollar2dp helper methods on show page to number_to_currency EX…
3986b09 
…ECPT in the plege field
@cece24
Modify Reward claim action to check for reward limit, compare to clai…
8c40b96 
…med, create pledge and save
@cece24
Merge pull request #5 from joshdales/cece-oct-26
57e2203 
Cece oct 26
@achristens
New User form validation errors displayed
58b81af
@achristens
New Project form validation errors displayed
e8c6976
@demileee
Updated seeds to create updates, link projects and pledges to users
03dcb04
@demileee
Add updates to project show page
6e10759
@demileee
Merge branch 'master' of github.com:joshdales/crowdfunder
33d95fc
@achristens
Display errors for login and pledge forms
13df260
@achristens
Merge pull request #6 from joshdales/abbyoct27
7464b77 
Abbyoct27
@demileee
Complete edit,update,destroy actions for project_updates
069637a
@demileee
Merge branch 'master' of github.com:joshdales/crowdfunder
c5bdbac
@demileee
update pledge form to not show if you are project owner, update homepage
408faee
@demileee
Project pledges sum move as helper method, formatted plural/single words
c314650
@demileee
Add back dollar_2dp helper method, show who pledged on project show
cd69158
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@demileee @achristens @joshdales @cece24