Skip to content
This repository has been archived by the owner on May 23, 2022. It is now read-only.

Stand Alone Widget Deployment #145

Merged
merged 16 commits into from
Jun 17, 2019
Merged

Stand Alone Widget Deployment #145

merged 16 commits into from
Jun 17, 2019

Conversation

mlewis22
Copy link
Contributor

@mlewis22 mlewis22 commented Jun 8, 2019

What?
Widget index.js has been improved. In addition to its current properties, developers can use this generator to create widgets in any folder location. This allows developers the flexibility to use the centralized widget templates without the extra requirement of having additional environmental setup such as a preconfigured task runner.

How?
When a user runs
yo esri-appbuilder-js:widget

The code assess the current root project for a .yo-rc.json file. If it finds one it will continue to use the predefined properties currently utilized in the generator. If it doesn't, it will ask the user for a place to store the widgets. It will then offer the same options to the user when presented with initial project creation.

Why?
This use case has been generated to allow the creation of widgets inside a project that utilities docker and gulp. The project wanted to use the benefits of the generator without the restrictions of the initial project outlay. I felt it would be a useful low impact addition.

@gavinr
Copy link
Contributor

gavinr commented Jun 12, 2019

thanks for the contribution @mlewis22!

  1. It looks like you've included a commit that includes the "Theme generator" in this PR. Could you please modify this branch (or maybe create a new branch/PR) that contains only the code changes related to this PR (Stand Alone Widget Deployment)
  2. Looks like there are some unit test failures. Please run npm run test on the code and either fix the failures or update the unit tests whee applicable.

@mlewis22
Copy link
Contributor Author

Sorry - I thought i was clever and moved it into a different branch. I forgot about the test! (Bad person). Ill take a look and put a new pull.

@mlewis22
Copy link
Contributor Author

@gavinr -- updates to the code and readme so tests are now passing

Copy link
Contributor

@gavinr gavinr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@gavinr gavinr merged commit 6ddcbbb into Esri:master Jun 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants