Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Disconnect PayPal gives "Looks like you've found a bug! Sorry for the inconvenience, we'll get it fixed ASAP!" #4168

Closed
toy opened this issue Oct 29, 2016 · 11 comments
Assignees
Labels

Comments

@toy
Copy link

toy commented Oct 29, 2016

Account toy, I have only one verified email, when clicking Disconnect button on PayPal page, I get the error.

@chadwhitacre
Copy link
Contributor

Sorry, @toy. :-/

Don't have time now, but I will look into this when I get a chance if someone else doesn't get to it first ...

@toy
Copy link
Author

toy commented Oct 29, 2016

It is not really urgent. I already saw this bug, but assumed that you are getting notifications, but as the bug was not fixed, I've created an issue.

@rohitpaulk
Copy link
Contributor

Copying the error from Sentry:

IntegrityError: update or delete on table "exchange_routes" violates foreign key constraint "exchanges_route_fkey" on table "exchanges"
DETAIL:  Key (id)=(10189) is still referenced from table "exchanges".

Looks like we're trying to delete the exchange route, but the foreign key constraint between exchanges -> exchange_routes prevents us from doing so.

Hmm, maybe we should be soft deleting exchange routes?

@rohitpaulk rohitpaulk added the Bug label Oct 29, 2016
@chadwhitacre
Copy link
Contributor

assumed that you are getting notifications

We are getting notifications, but we're not staying on top of them. Sorry. 😞

@rohitpaulk rohitpaulk self-assigned this Oct 31, 2016
@rohitpaulk
Copy link
Contributor

Aha!

screen shot 2016-10-31 at 6 37 24 pm

@rohitpaulk
Copy link
Contributor

rohitpaulk commented Oct 31, 2016

The DELETE statement was added by me in 0500105, and @whit537 added the comment above that in e27a228.

I think the best way out here is to update the error to 'invalidated' irrespective of the network - and when user tries to add the route again for paypal, we revive the previous exchange_route.

@rohitpaulk
Copy link
Contributor

Moving implementation details to a PR so that we don't pollute this issue

@toy
Copy link
Author

toy commented Jan 9, 2017

Still not possible to disconnect

@chadwhitacre
Copy link
Contributor

Mind giving this another shot, @toy? We just deployed #4170 in an attempt to fix this.

@toy
Copy link
Author

toy commented Mar 2, 2017

@whit537 Thanks! It seems to work, as I was able to disconnect the paypal account.

@toy toy closed this as completed Mar 2, 2017
@chadwhitacre
Copy link
Contributor

Woo-hoo! Thanks for your patience. :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants