Skip to content
This repository has been archived by the owner on Feb 16, 2020. It is now read-only.

include module exports in instructions #2862

Closed
wants to merge 2 commits into from

Conversation

grenade
Copy link

@grenade grenade commented Oct 20, 2019

in issues #2493 and #2576, users saw an error (TypeError: Cannot read property 'timeout' of undefined) that is not easy to understand because they had edited their uiconfig files according to the instructions here, which omit the export. This change is to include an example of a complete uiconfig file to reduce confusion by users who copy and paste the example code over the entire contents of their existing uiconfig.

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

    • docs update
  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)?

    • the user is presented with a complete example of uiconfig, preventing errors of omission.
  • Other information:
    none

in issues askmike#2493 and askmike#2576, users saw an error (`TypeError: Cannot read property 'timeout' of undefined`) that is not easy to understand because they had edited their uiconfig files according to the instructions here, which omit the export. This change is to include an example of a complete uiconfig file to reduce confusion by users who copy and paste the example code over the entire contents of their existing uiconfig.
@stale
Copy link

stale bot commented Dec 20, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If you feel this is very a important issue please reach out the maintainer of this project directly via e-mail: gekko at mvr dot me.

@stale stale bot added the wontfix label Dec 20, 2019
@stale stale bot closed this Dec 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant