Issue 401 #557
Issue 401 #557
Conversation
…for role validation
| @@ -0,0 +1,5 @@ | |||
| class RemoveRoleIdFromUser < ActiveRecord::Migration[7.0] | |||
| def change | |||
| remove_column :users, :role_id, :integer | |||
There was a problem hiding this comment.
O rollback aqui está errado. Tive um problema de validação na migration seguinte (por ter essa parte de usuários bem bagunçada em desenvolvimento) e não consegui dar rollback para uma versão funcionando.
Só restaurar a coluna role_id não é o suficiente: é preciso pegar de UserRole
| @@ -137,6 +139,13 @@ | |||
| concerns :active_scaffold | |||
| end | |||
|
|
|||
| resources :user_roles do | |||
| concerns :active_scaffold | |||
There was a problem hiding this comment.
Esse concerns faz sentido? Não tem o bloco do active scaffold no user roles...
Por consistência, acho que seria bom adicionar o bloco lá (ao invés de tirar o concerns daqui), mesmo que a rota pra user_roles não fique visivel no menu de navegação
| @@ -41,7 +41,10 @@ | |||
| <% end %> | |||
| </div> | |||
| <div id="sub"> | |||
| Versão <%= APP_VERSION %> | <%= link_to 'Créditos', credits_show_path %> | |||
| Versão <%= APP_VERSION %> | <%= link_to 'Créditos', credits_show_path %> | |||
| <div id="role_selector" style="padding-left: 90px;"> | |||
There was a problem hiding this comment.
Será que não dá pra ajustar esses elementos para deixar o combo colado na direita? Provavelmente tem que fazer outros ajustes também, como deixar o campo de texto que está solto com a versão em uma div, mas me parece algo que dá pra resolver com flex do css
app/models/user.rb
Outdated
| scope :professors, -> { where(role_id: Role::ROLE_PROFESSOR) } | ||
| scope :coordination, -> { where(role_id: Role::ROLE_COORDENACAO) } | ||
| scope :secretary, -> { where(role_id: Role::ROLE_SECRETARIA) } |
There was a problem hiding this comment.
Essas consultas estão quebradas. Elas são usadas na parte de Seleção -> Comitês -> Adicionar com os links "Adicionar todos [professores|coordenação|secretaria]"
app/models/enrollment.rb
Outdated
| user = User.invite!({ | ||
| email: student.first_email, | ||
| name: student.name, | ||
| role_id: Role::ROLE_ALUNO | ||
| actual_role: Role::ROLE_ALUNO | ||
| }, current_user) do |invitable| | ||
| invitable.skip_confirmation! | ||
| end |
There was a problem hiding this comment.
Esse usuário está sendo criado sem papel em UserRole. Só o actual_role está sendo preenchido
…es, addition of a new hidden page /user_roles
…tion related to user and professor/student
…cations of this Issue
No description provided.