Melhorias registro de conteúdos #126
Conversation
|
@edersoares, tudo bem? Como conversamos, está ai a melhoria que discutimos a respeito! @gustavomendess, acabei realizando o PR sem passar por uma apresentação na Comunidade da nova funcionalidade pois se fosse esperar isso demoraria muito, estou bastante ocupado! |
UP! 🚀 |
|
@marcosoliveirasoares94 obrigado pela contribuição, vamos fazer as análises internas de compatibilidade e retornamos ;) |
@edersoares, complementando este PR com esse novo PR #128 |
There was a problem hiding this comment.
@marcosoliveirasoares94 @gustavomendess muito obrigado pela contribuição!
Desculpem a demora, estaremos adicionando no nosso fluxo de qualidade para aprovarmos o merge ;)
| <td width="160px"></td> | ||
| </tr> | ||
| </thead> | ||
|
|
||
| <%= render 'resources' %> | ||
|
|
||
| <tfoot id="paginator" class="remote"> | ||
| <td id="pagination-tfoot" colspan="5"> | ||
| <td id="pagination-tfoot" colspan="6"> |
There was a problem hiding this comment.
Colspan vai depender se a @allow_class_number for true
| @@ -0,0 +1,5 @@ | |||
| class AddClassNumberToContentRecord < ActiveRecord::Migration | |||
| def change | |||
| add_column :content_records, :class_number, :integer, default: 0 | |||
There was a problem hiding this comment.
A coluna pode ser nullable e nil ser o default
script/start
Outdated
| bundle exec rake entity:setup NAME=prefeitura DOMAIN=localhost DATABASE=prefeitura_diario | ||
| bundle exec rake entity:admin:create NAME=prefeitura ADMIN_PASSWORD=123456789 | ||
| bundle exec rake entity:setup NAME=prefeitura DOMAIN=localhost DATABASE=idiario | ||
| bundle exec rake entity:admin:create NAME=prefeitura ADMIN_PASSWORD=A123456789$ |
There was a problem hiding this comment.
Não faz sentido mudar isso neste PR ;)
README.md
Outdated
| Acesse o endereço que será exibido após rodar o comando acima. | ||
|
|
||
| O usuário padrão é: `admin` / A senha padrão é: `123456789` | ||
|
|
There was a problem hiding this comment.
Não faz sentido mudar isso neste PR ;)
|
@edersoares e @gustavomendess algum retorno sobre essa melhoria proposta pela comunidade, ela está a bastante tempo aguardando reintegração e é algo muito útil e parametrizável? O ingresso dessa funcionalidade estimula a contribuição por parte da comunidade, pois sabemos que nossas propostas são aceitas. |
|
@msoares94 @gustavomendess eu precisei atualizar esta branch para retomarmos essa melhoria, conseguem validá-la novamente por favor? Principalmente este trecho https://github.com/portabilis/i-diario/pull/126/files#diff-931228249692a941a48c22abe2b692ed5d6cdd4729f053fd78ea63b93485034cR81-R89. |
|
Já aceito o PR msoares94#1. |
Closes #119
Possibilita criar mais de um conteúdo para o mesmo dia, caso o parâmetro esteja ativo em Administrativo > Configurações gerais, ele passa a validar as aulas e não o dia.
Ao desabilitar o parâmetro, a validação volta a ser por dia/disciplina e as colunas de aula não aparecerão.
Para ativar o parâmetro:
url/configuracoes-gerais/editarNo checkbox:
Após ativar o check acima, na listagem dos registros já irá aparecer o número de aula na listagem.
Ao entrar pra criar um registro, aparecerá o campo para selecionar a aula.
Neste exemplo, ao selecionar as 3 aulas, a criação irá triplicar os conteúdos e criar para as 3 aulas.
Caso haja alguma aula utilizada, irá retornar a mensagem de erro dizendo que a aula está em uso.
Ao visualizar um conteúdo criado, o campo aula deverá ficar desabilitado:
Ao editar, a aula também não poderá ser ajustada, apenas os conteúdos:
IMPORTANTE