Commit Graph

339 Commits (36df27cbc2a1949bc1f25512631d1b880b4bfc96)

Author SHA1 Message Date
eric 6539370e37 remove commented code 2013-11-25 13:37:15 -05:00
eric dd83434282 Fix error handling in Pay 2013-11-25 13:01:16 -05:00
eric 18f104c210 latent spelling error 2013-11-24 18:44:40 -05:00
eric 1c33c34f21 purchase complete notification, contextual wishlist active, fulfillment in signal handler 2013-08-22 14:30:56 -04:00
eric 75ec13cb39 make sure the transaction isn't double charged! 2013-08-22 14:03:22 -04:00
eric 80d78e624f remove amazon cruft to get current_total correct 2013-08-19 23:23:27 -04:00
eric 91f7c5f23d Remove some payment dependence on campaign details 2013-08-19 22:52:22 -04:00
eric 8317356cc2 moved to not_maintained 2013-08-19 22:27:25 -04:00
eric d5b9f88380 move "extra data" into a json field dict to improve payment code tidieness
need to install jsonfield
note: this is https://bitbucket.org/schinckel/django-jsonfield
I tried https://github.com/bradjasper/django-jsonfield but it was not
compatible with South
2013-08-18 15:21:55 -04:00
Raymond Yee f9fad3260a instance -> self in Account.update_status() -- d'oh 2013-08-08 16:28:34 -04:00
Raymond Yee cf7160b72d Adding an optional flag to Account.update_status, which by default, send notices only if the status is *changing*.
Set send_notice_on_change_only = False to send notice based on new_status regardless of old status.  (Useful for initialization)
2013-08-08 13:31:38 -04:00
eric c30c0de604 make recharge_failed_transactions a method on account
There were four places in the code that triggered the
recharge_failed_transactions signal and only one of them mattered. I
think this layout makes the code much easier to understand.
2013-08-05 14:37:46 -04:00
eric 25d9edf4d8 fix non-working update_status method 2013-08-05 12:18:18 -04:00
eric b28412ec7a fix import on payment/tasks 2013-08-05 12:15:04 -04:00
eric 6da8d16770 moved the payment tasks to payment from core 2013-08-03 18:42:33 -04:00
Raymond Yee 3936692244 [#40140123] Add Account.update_status() to replace handle_Account_status_change thus resolving a number of issues Eric identified:
* [ ]https://github.com/Gluejar/regluit/pull/176/files#r4221450 "It also seems odd to be using signals to talk only within the same module. This seems to me to be a misuse of signals, and makes for code that is harder to read"  -- I need to decide whether I agree w/ Eric.
   * [ ]https://github.com/Gluejar/regluit/pull/176/files#r4366503 long exchange "making it readonly would be overkill (java-ish) but the current code is not clear. Your save signal handler has to figure out whether a state transition has occurred and then act accordingly, which seems like a lot of hoops to jump though. better to put all the transition work in one place. For example, shouldn't it be the deactivate method that sets the status to deactivated?

For example, the methods on campaign do a reasonable job of handling all the status transitions."  What I  should:  mimic structure of  4fc449dad5/core/models.py (L322)   -> time to write up how Campaign works

   * [ ]https://github.com/Gluejar/regluit/pull/176/files#r4214827 "could also mean the Account has been deleted. don't you really want instance.pk=None?"

   * [ ]https://github.com/Gluejar/regluit/pull/176/files#r4221509 "since, these actions are being done in batch mode, the emit notifications would work a lot better if they were emitted after the batch of notifications is created."
2013-07-23 14:16:42 -07:00
Raymond Yee 37ce78fda2 Merge branch 'master' into expiring_cc 2013-07-17 18:12:30 -07:00
Raymond Yee dabaa2f843 update tests to account for new events (now stripe allows multiple cards/customer and now generates card related events in addition to customer events) 2013-07-15 14:52:29 -07:00
Raymond Yee e4ab0ccd61 Merge branch 'instant_charge' into expiring_cc 2013-07-09 14:46:06 -07:00
Raymond Yee 46d1d4f767 Merge branch 'master' into expiring_cc
Conflicts:
	payment/models.py
	payment/tests.py
2013-07-08 10:52:44 -07:00
Raymond Yee ccbc78a10c added a first pass for payment.tests.TransactionTest.test_paymentmanager_charge to illustrate use of PaymentManager.charge() -> hardwired for stripelib right now.... 2013-06-28 16:05:19 -07:00
Raymond Yee 3ff53dd5bc first pass at stripelib.Processor.Pay 2013-06-26 18:46:30 -07:00
Raymond Yee d3166d3aef Merge branch 'master' into instant_charge 2013-06-26 17:17:07 -07:00
eshellman 5a207bf4e8 Merge pull request #181 from Gluejar/simpler_download
Simpler download. Yay.
2013-06-26 11:39:39 -07:00
Raymond Yee 701533117f fixed some seemingly bad defaults in baseprocessor
basic pass at payment.manager.PaymentManager.charge
2013-06-25 15:34:20 -07:00
Raymond Yee 867e745f75 Merge branch 'master' into instant_charge 2013-06-25 11:26:39 -07:00
Raymond Yee cae7f6d862 also delete the commented out pledge method 2013-06-25 11:23:21 -07:00
Raymond Yee 22dc844b07 Getting rid of tests that call PaymentManager.pledge 2013-06-25 11:05:52 -07:00
Raymond Yee 1725331aa0 cleaning out some old refs to Paypal 2013-06-25 10:39:49 -07:00
Raymond Yee 5e3fca0282 in stripelib.py, I am resurrecting PledgeScenarioTest
in manager.py, a very rough first pass at charge -- not working yet, but doesn't break things either
2013-06-24 16:19:12 -07:00
Raymond Yee 53178d80ce moving to stripe 1.9.1 2013-06-21 15:14:51 -07:00
Raymond Yee a36ff38745 extraneous creation of StripeClient 2013-06-20 13:39:06 -07:00
Raymond Yee c76a320912 * Comment out current payment.manager.PaymentManager.pledge -- tests run ok -- so it looks like I suspect: that we don't use that code and it's a hold-over from PayPal code
* stub for stripelib.Processor.Pay
2013-06-19 18:54:32 -07:00
Andromeda Yelton ad69580b2d PEP 8, PEP 328. I feel better now. 2013-06-03 12:31:39 -04:00
Raymond Yee ab371c0e64 turn the payment.Account.status into an enumeration (with choices) 2013-05-28 15:57:22 -07:00
Raymond Yee a3cc502924 get rid of AccountTest0
renamed notice to Credit Card Number Updated
actually save account in update_account_status
2013-05-20 13:58:21 -07:00
Raymond Yee ef8c27d6fa Fire off account_active notices only if Account is a new one 2013-05-09 08:54:00 -07:00
Raymond Yee 94a23b3adf delete AccountTest.get_transaction_level() 2013-05-08 19:03:32 -07:00
Raymond Yee b1a23c9183 Add sending of account_active notices -- though they are turned off by default 2013-05-08 18:15:33 -07:00
Raymond Yee db1ee70647 fixed up notice texts for account_expired
also made sure that user is properly passed to sender of notices
2013-04-26 10:50:28 -07:00
Raymond Yee a3127e8086 Trying to work around problems in old AccounTest 2013-04-12 11:13:26 -07:00
Raymond Yee 9cc2e3121e Merge branch 'master' into expiring_cc 2013-04-10 09:14:29 -07:00
Raymond Yee ed37463f35 debugging tests on ry-dev -- let's see how this runs on ry laptop 2013-03-20 21:56:37 +00:00
Raymond Yee aca38d66a1 Merge branch 'master' into expiring_cc
Conflicts:
	payment/tests.py
2013-03-20 09:41:30 -07:00
eric af8c3ff1fd merge from master 2013-03-19 10:02:06 -04:00
eric d3077c988b links should not point to http from https 2013-03-09 13:29:03 -05:00
eric 800d0ffdeb cleanup of css and js imports, also mapping 2013-03-08 14:16:24 -05:00
eric d6a8dedfaa mailchimp ops now example.org aware, test users, now all changed to example.org 2013-03-06 09:39:06 -05:00
Raymond Yee de96aa0755 Add task for 22nd of month for notifying expring cc 2013-03-04 11:19:13 -08:00
Raymond Yee 0174ef863f card expiring and expired notices firing now 2013-03-04 09:06:57 -08:00