Closed
Bug 782710
Opened 13 years ago
Closed 13 years ago
Remove pre-approval tokens
Categories
(Marketplace Graveyard :: Payments/Refunds, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
2012-09-06
People
(Reporter: andy+bugzilla, Assigned: oremj)
Details
We aren't using the PayPal pre-approval tokens in the marketplace since we aren't using PayPal in the next push. We should just remove any existing tokens just to be sure we've got nothing lying around. If PayPal gets turned back on, I don't think it will be a big issue for those users to set it up again.
| Reporter | ||
Comment 1•13 years ago
|
||
https://github.com/andymckay/zamboni/commit/388782
Removed permissions tokens as well.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 2•13 years ago
|
||
Since this was reverted, we should probably keep the bug open. yes?
Comment 4•13 years ago
|
||
What happened with this bug? I thought this was about deleting data - how would it be reverted?
Target Milestone: 2012-08-16 → 2012-08-30
| Reporter | ||
Comment 5•13 years ago
|
||
The SQL statements are there, but were never run on prod. There is no point in running this until we've got the PayPal code all removed / flagged off. Otherwise people will just set them up again. When the PayPal code is removed / flagged off, run the SQL code in the migration and the jobs done. So we'll just keep bouncing this along until then.
Comment 6•13 years ago
|
||
Kumar - is the payments stuff flagged off enough to run this?
Target Milestone: 2012-08-30 → 2012-09-06
Comment 7•13 years ago
|
||
Yeah, all payments and related pages are disabled.
| Reporter | ||
Comment 8•13 years ago
|
||
Just moved migration so no-one has to do anything manually:
https://github.com/mozilla/zamboni/commit/8a3c9b
Assignee: amckay → oremj
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment 9•13 years ago
|
||
Verified as fixed...all pages related to payments are disabled.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•