angular-fullstack/generator-angular-fullstack

Should client/app/admin and account be full components?

Open

#2.490 aberto em 4 de fev. de 2017

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (1.268 forks)batch import
Help Wanted

Métricas do repositório

Stars
 (6.134 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

  • I understand that GitHub issues are not for tech support, but for questions specific to this generator, bug reports, and feature requests.
Item Version
generator-angular-fullstack 4.1.2
Node 6.9.4
npm 4.1.2
Operating System OS X 10
Item Answer
Transpiler Babel
Markup Pug
CSS SCSS
Router ui-router
Client Tests Mocha
DB Sequelize 4.0.0-2 / PostgreSQL 9.6.1
Auth Y

account(auth) and admin(auth) are implemented as loosely coupled controllers. Their implementation differs from the generated main component (which is an AngularJS component). Also, possibly change their controllerAs to $ctrl from vm.

What are your thoughts about making admin, login, settings, and signup into full AngularJS components? This would improve consistency as well as lay the foundation for those moving to 2.0.

Guia do colaborador

Direção de pesquisa
Refatore as subaplicações account e admin de controladores para componentes AngularJS, seguindo o padrão do componente principal. Atualize controllerAs para $ctrl e garanta consistência. Considere também o caminho de migração para o Angular 2.0.
Pilha de tecnologia
javascript
Domain
full stack
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
AngularJSYeoman
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65