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

feat(build, helpers): issues/52 debug config params #69

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Aug 2, 2019

What's included

  • feat(build, helpers): issues/52 debug config params
    • build, dotenv logger id
    • helpers, debugging access to open dotenv config params, via window
    • contributing md updated to explain debugging use

Notes

  • 😄 like an about modal without the about modal!

How to test

Coverage and basic unit test check

  1. update the NPM packages with $ yarn
  2. $ yarn test

Double check an environment

  1. update the NPM packages with $ yarn
  2. $ yarn start
  3. open the browser console with the GUI focused, type in curiosity or window.curiosity you should receive an object containing strings and functions

Example

Screen Shot 2019-08-02 at 12 14 54 PM

Updates issue/story

Updates #52

* build, dotenv logger id
* helpers, debugging access to dotenv config params, via window
* contributing md updated to explain debugging use
@cdcabrera cdcabrera changed the title feat(build, helpers): issues/52 expose basic config params feat(build, helpers): issues/52 debug config params Aug 2, 2019
@cdcabrera cdcabrera requested a review from priley86 August 2, 2019 16:21
@cdcabrera cdcabrera merged commit 2760ab2 into RedHatInsights:master Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants