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

unify client services #65

Open
galenriley opened this issue Feb 8, 2020 · 4 comments
Open

unify client services #65

galenriley opened this issue Feb 8, 2020 · 4 comments
Assignees

Comments

@galenriley
Copy link
Member

Bring login into the axios pattern
on creating a post, first check that we are logged in and try to login again if not

@tacoman
Copy link
Member

tacoman commented Jun 15, 2020

Login will be brought into the Axios pattern by #118

The post check will require slightly more work.

@galenriley
Copy link
Member Author

Okay with closing this Issue once the PR is done/merged, and then we can create a new one for checking login is valid before applying the post. I forget the inspiration for that element of this issue

@galenriley
Copy link
Member Author

oh, was it a login timeout thing?

@galenriley
Copy link
Member Author

There's one more fetch() call that should be migrated to axios- the push token registration in /containers/GlobalDataContainer.js

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

No branches or pull requests

2 participants