angular / angular-cli

CLI tool for Angular
https://cli.angular.dev
MIT License
26.77k stars 11.98k forks source link

Feature: "ng generate component" needs a flag to generate routes for standalone components #27201

Open stephenwithav opened 8 months ago

stephenwithav commented 8 months ago

Command

generate

Description

Running ng generate component users --standalone generates users/users.component.{css,html,spec.ts,ts} files.

For imported or lazy-loaded routes, we can create a users/users.routes.ts that looks like this:

import { Routes } from '@angular/router';
import { UserListComponent, UserDetailsComponent, CreateUserComponent, EditUserComponent } from './users/users.component';

export const routes: Routes = [
    { path: 'list', component: UserListComponent },
    { path: 'details/:id', component: UserDetailsComponent },
    { path: 'create', component: CreateUserComponent },
    { path: 'edit', component: EditUserComponent },
];

...and import it with loadChildren in app.routes.ts, but ng could simplify the process.

Describe the solution you'd like

ng generate component users --standalone --routes could scaffold the users/users.routes.ts file for us, letting us just fill it in as we go.

import { Routes } from '@angular/router';
import * from './users.component';

export const routes: Routes = [
    { path: '', component: UsersComponent },
];

An alternative is ng generate component users --standalone --routes=crud, which generates components and route mappings for each CRUD component. ng generate component users --standalone --routes=cr would only generates Create and Read components and mappings.

Describe alternatives you've considered

yasnippets with emacs, but ng incorporating this functionality would make life easier.

stephenwithav commented 8 months ago

@alan-agius4 Would you like me to take a shot at adding this functionality?

alan-agius4 commented 8 months ago

This is actually something that @AndrewKushnir was looking at sometime ago. We still had some pending design decisions, which required broader discussions with the rest of team.

stephenwithav commented 8 months ago

Thank you for the info. I'll implement a placeholder in elisp for now.

@AndrewKushnir, I'm here to help if you want when those design decisions are made.