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.

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

Attempt to update deps (does not work), see description #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phoenixy1
Copy link
Collaborator

@phoenixy1 phoenixy1 commented Dec 2, 2024

Yes, I know, I accidentally committed my authtoken for ngrok. I rotated it and the authtoken in this PR is no longer valid.

Anyway, I attempted to update the dependencies (following the checklist here plaid/pattern#317) but after doing so repeatedly hit the client error "cannot use 'in' operator to search for validatestatus" on the client when triggering an ITEM_LOGIN_REQUIRED error.

I am fairly puzzled on this as it seems to be caused by sending a malformed post or delete request via Axios, but the code used by Pattern for sending Axios post requests to the Pattern server appears to be correctly formatted and this error apparently happens regardless of whether I update the Axios version that is used. The regular (transactions-based) pattern app also has exactly the same code for handling ITEM_LOGIN_REQUIRED and does not have this problem after updating dependencies.

Anyway, I spent a few hours on this and then I give up. I am just submitting this in case anyone has any brilliant ideas for fixing the "cannot use 'in' operator to search for validatestatus" error. Please do not spend much time (more than 30-60 minutes) on this; it isn't worth it.

@phoenixy1 phoenixy1 added help wanted Extra attention is needed invalid This doesn't seem right labels Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant