Closed solkaz closed 7 years ago
Why did you refactor all the gists stuff?
@thebho part of refactoring the CodesplainAppBar, so that most of the logic is abstracted away.
This "works", but I cannot figure out a way to test this. Merge at your own discretion
Do you mean you don't have a way to write unit tests, or you have never actually seen the code work?
I don't have a way to write unit tests. I can verify that this works in all browsers, though
Description
When a user attempts to access their information with an invalid Github token, the application will clear their credentials and reset the application. This will occur early in the application life cycle, specifically in
<CodesplainAppBar />
'scomponentDidMount
method.Abstracts much of the promise logic away from the component, and instead lets the thunks handle them.
This PR relies on this PR being merged in order to work correctly
Motivation and Context
Fix #465
Checklist:
Browsers