Code Monkey home page Code Monkey logo

Comments (8)

angelozerr avatar angelozerr commented on June 12, 2024

Is there any change to have a PR with wizard page which contains thoses options?

from angular-eclipse.

probert94 avatar probert94 commented on June 12, 2024

I am planing to add this Options to the New-Project WizardPage, just like I did with New Module Wizard Page, if thats okay.
If there are too many options, I might add another WizardPage...

from angular-eclipse.

angelozerr avatar angelozerr commented on June 12, 2024

IF there are too many options, I might add another WizardPage...

IMHO, I think we should have 2 pages:

  • a page with requiered fields (the project name) like today. And add in this page the angular-cli version.
  • a second page with optional parameters. Those optional parameters could be dynamic according the angular-cli version. The idea is to execute ng help with the given angular-cli (to retrieve those options) and generate an UI (a checkbox if it's a boolean, etc). But perhaps we could start with a simply mean and hard coded the optional fields.

from angular-eclipse.

probert94 avatar probert94 commented on June 12, 2024

a second page with optional parameters.

A second page sounds good. Do you think the (future) optional parameters for the "generate" commands should be on a second page to?

Those optional parameters could be dynamic according the angular-cli version. The idea is to execute ng help with the given angular-cli (to retrieve those options) and generate an UI (a checkbox if it's a boolean, etc). But perhaps we could start with a simply mean and hard coded the optional fields.

The installed angular-cli version could really cause some problems.
In the case of the New-Project-Wizard, we have the possibility to consume the "ng help" command (as long as they don't change the structure of the answer).
However, the help of the "generate" command shows only a few general params (like "dry-run").
Other, blueprint specific, parameters aren't documented (on the Github page they are also a bit hidden and can be found here).

For now maybe hardcoding the optional fields will do (lets say we define a min. and max. cli-version for each option).
However it could cause problems, if they remove an option, as the plugin might be older then the cli-version...

from angular-eclipse.

angelozerr avatar angelozerr commented on June 12, 2024

Do you think the (future) optional parameters for the "generate" commands should be on a second page to?

I think we could keep the optional options in the same page since this page shows fews fields. We could hide thoses fields by default and display a label like "Optional parameters". When user click on "Optional parameters", it expands and shows optional parameters.

For now maybe hardcoding the optional fields will do

Agree with you. It should be cool if we could comment each field (available since version X)

from angular-eclipse.

probert94 avatar probert94 commented on June 12, 2024

@angelozerr I just tryed to use unknown parameters, when generating a new project and saw, that Angular-CLI just brings a warning and ignores the parameter.
So I think it is not really neccessary to store the Min-CLI-Version required for a parameter, or what do you think?

from angular-eclipse.

angelozerr avatar angelozerr commented on June 12, 2024

So I think it is not really neccessary to store the Min-CLI-Version required for a parameter, or what do you think

I think it should be more cleaner to show in the UI wizard page just the available parameters for an angular cli version. But for the moment don't loose our time with that, but I think it should be cool if we could just comment before the declaration of the field "avilable since $version. In the future we could use those comments to improve the UI and show only available parameters.

from angular-eclipse.

probert94 avatar probert94 commented on June 12, 2024

Yea you are right, it's definitely cleaner, but not required for the moment.
I am almost odne with this, the project generation using all the different parameters, seems to work.
I only need to rearrange the Widgets on the WizarPage, then I will commit the changes.

For the currently available parameters I would prefer to say that beta 25 or later is required, instead of looking through the whole Angular-CLI history, to perfectly support versions older then beta 25 ("skip-tests" for example is availlable since beta 25).

from angular-eclipse.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.