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

Fix react_client reference in README #18

Merged
merged 1 commit into from
Oct 31, 2016

Conversation

johncblandii
Copy link
Contributor

Ultimate problem:

The README code does not run.

How it was fixed:

Named the import react_client so the example code runs.

Testing suggestions:

Paste code and verify it runs.

Potential areas of regression:

None.


FYA: @greglittlefield-wf @aaronlademann-wf @jacehensley-wf @clairesarsam-wf @joelleibow-wf

The README code does not compile because it looks for react_client.
#16 is needed to make the code actually run as well.

The README code does not compile because it looks for `react_client`.

Workiva#16 is needed to make the code actually run as well.
@aviary-wf
Copy link

Raven

Number of Findings: 0

@codecov-io
Copy link

Current coverage is 97.03% (diff: 100%)

Merging #18 into master will not change coverage

@@             master        #18   diff @@
==========================================
  Files            27         27          
  Lines          1211       1211          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1175       1175          
  Misses           36         36          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 11ac5f5...1f3b952

@aaronlademann-wf
Copy link
Contributor

+1

@colefeisthamel-wf
Copy link

RM +1 manually reviewed dependencies

greglittlefield-wf added a commit that referenced this pull request Jun 19, 2020
Update addRef assist / add createRef quick fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants