Commit Graph

78 Commits (958d963f7543d5e9bfd30373ab0d743a1a724644)

Author SHA1 Message Date
eric f7fea1874e wish list was not getting set because of vestigial transaction type check
[fix #67399604]
2014-03-14 13:04:11 -04:00
eric 678eb02847 Thanks works with anonymous user 2014-02-20 15:56:10 -05:00
eric b5fd630bfc Thanks mostly works. 2014-02-19 22:18:23 -05:00
eric b3465ea084 search code for dependence on deadline being not null 2014-01-15 17:59:45 -05:00
eric b331866c38 fix test 2013-12-15 01:26:42 -05:00
eric 40e611bac5 revive the gift/donation credit funtionality, at least for b2u campaigns
The reason for doing this is so a library can put in money with one
check, then use it up gradually. (we get to pocket the fees that would
go to stripe) One change here- in b2u there is an extra click to get to
download page, even if no gift credit is involved - this way the user
knows what and how he's bee charged
2013-12-15 00:31:06 -05:00
eric 2892a22824 change terminology from "donation" to "gift" 2013-12-13 15:15:35 -05: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
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 46d1d4f767 Merge branch 'master' into expiring_cc
Conflicts:
	payment/models.py
	payment/tests.py
2013-07-08 10:52:44 -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 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 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
Raymond Yee a7f6d8e7cc put more of the skeleton for Account pre_save handler 2013-03-01 17:54:37 -08:00
Raymond Yee 76a88e2db2 placed a pre_save handler to provide a part of framework for Account.status change 2013-02-27 11:31:02 -08:00
Raymond Yee cbfabbfa3c Implemented Account.status migration 2013-02-27 10:08:36 -08:00
Raymond Yee c3f922e9ba First pass at regluit.payment.models.Account.status 2013-01-14 16:18:19 -08:00
Andromeda Yelton be6a1082c2 whoa, urlencode takes dicts and this was failing silently all along 2012-12-11 13:07:09 -05:00
Raymond Yee ce6e7f865d tests pass on this commit 2012-11-20 13:56:14 -08:00
Raymond Yee f2bb53f132 recharge signal being fired now 2012-11-12 12:48:03 -08:00
Raymond Yee a90f133166 First pass at writing a signal handler to recharge transaction if account updated 2012-11-12 11:22:20 -08:00
Raymond Yee 30fd1332af Expanding functionality of stripelib.Processor.Execute -- better error handling 2012-10-31 08:23:13 -07:00
eric bda3211a76 user can manage their account credit card [#37674369] 2012-10-14 23:41:17 -04:00
Raymond Yee b904cb1f28 Fixed a bug in the logging statement that had mismatched # of parameters 2012-10-13 08:23:07 -07:00
eric 2ae6942f4f better to not have logging of tokens 2012-10-13 00:48:46 -04:00
eric 476ce46dab display and notifications of acknowledgements 2012-10-12 23:45:09 -04:00
eric 6a1646f749 fixed bad method signature 2012-10-12 22:25:48 -04:00
Raymond Yee b281aca869 [#37567927] enforce constraint of at most 1 active Account / user of given host
-> migration was missing before from an earlier commit -- add in now.
2012-10-11 16:36:24 -07:00
eric 73910ae8a0 To get credit module working again, had to move some methods into an object that inherits from a base class
So now all payment processor specific methods are on a Processor
object,
2012-10-02 23:49:19 -04:00
eric 7a1cf969ca fixed circular import problems [finish #37053797] 2012-10-02 22:14:40 -04:00
Raymond Yee ebb2144055 Responding to Eric's comment: yes Account.date_created should have auto_now_add not auto_add
I modified migration and model.
2012-09-18 08:16:12 -07:00
Raymond Yee 6a650affc8 Now recording pledge as such -- and not charging immediately.
running into exception at https://github.com/Gluejar/regluit/blob/relaunch_ry/payment/manager.py#L554-- have to start implementing a baseclass for stripelib to handle pledge modification
2012-09-17 16:56:07 -07:00
Raymond Yee 69e959a0c8 fleshing out of payment.models.Account 2012-09-17 14:55:28 -07:00
Raymond Yee fd1887ea44 [#36028929] Work to pull out relevant fields in stripe Customer and Charge model with goal of figuring out which to add to payment.models.{Account | Transaction} 2012-09-14 17:00:21 -07:00
Raymond Yee c545173793 First pass at payment.models.Account 2012-09-13 16:43:54 -07:00
eric 6762c7d3f2 not really supposed to use Decimal.NaN changed to Decimal('NaN') 2012-09-07 14:40:00 -04:00
eric 0c9d71e4d6 introduce PledgeExtra data object to simplify method calls 2012-09-07 09:46:38 -04:00
eric d63063b740 add missing notification for donation pledges 2012-09-06 13:27:34 -04:00
eric d14548f3af not sure how I missed this bug 2012-09-06 11:36:52 -04:00
eric 8eeadc99d3 woo hoo, credit and donation scaffolding works end to end [#21784207] 2012-09-06 01:01:17 -04:00
eric c20e6299bb off to a great start this morning [#16582483] 2012-08-31 11:05:00 -04:00