Foi criada o model ClientAccount com migretion e nova gem omniauth foi adicionada#10
Foi criada o model ClientAccount com migretion e nova gem omniauth foi adicionada#10AtanaelVmax22 wants to merge 41 commits intomainfrom
Conversation
criamos o arquivo docker para criar o banco de dados de desenvolvimento adicionou as configureaçoes do banco de dados de desenvolvimento no projeto
…base Adicionar configuração de banco de dados e dotenv
…o a validação no migration que não permite nomes nulos.
Configuração inicial do Devise: inclusão do Devise, criação do modelo User e geração da migração
…nt-entity Adicionar Entidade CorporateClient
Adição da Entidade Contact
…tion_fee sejam nao-negativos
Criando entidade endereco e suas validacoes.
…-token-auth correção do conflito da migração
Foi Criada o model e migration do contracte corrigido o conflito
Foi adicionado o model physical_client
Feature/create service
| @@ -0,0 +1,14 @@ | |||
| class ClientAccount < ApplicationRecord | |||
| # Include default devise modules. | |||
| devise :database_authenticatable, :registerable, | |||
| validates :email, presence: true, uniqueness: true, format: { with: /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i } | ||
| validates :password, presence: true, length: { minimum: 8 }, if: :password_required? | ||
|
|
||
| devise :database_authenticatable, :registerable, |
| devise :database_authenticatable, :registerable, | ||
| :recoverable, :rememberable, :validatable | ||
|
|
||
| include DeviseTokenAuth::Concerns::User |
|
|
||
| mount_devise_token_auth_for 'User', at: 'auth' | ||
|
|
||
| mount_devise_token_auth_for 'ClientAccount', at: 'auth' |
There was a problem hiding this comment.
As rotas do devise devem estar localizadas nas primeiras linhas do arquivo
Isso foi testado?
Se sim, como?
Existe um conflito ali no at: 'auth'
There was a problem hiding this comment.
não foi testado, apenas rodou aplicação no rails server
| class DeviseTokenAuthCreateClientAccounts < ActiveRecord::Migration[7.0] | ||
| def change | ||
|
|
||
| create_table(:client_accounts) do |t| |
There was a problem hiding this comment.
Existem dois create_table(:client_accounts)
| @@ -0,0 +1,42 @@ | |||
| class DeviseCreateClientAccounts < ActiveRecord::Migration[7.0] | |||
| def change | |||
| create_table :client_accounts do |t| | |||
There was a problem hiding this comment.
Existem dois create_table(:client_accounts)
| @@ -0,0 +1,14 @@ | |||
| class ClientAccount < ApplicationRecord | |||
There was a problem hiding this comment.
ClientAccount é um nome fora do padrão
Isso deveria ser algo com ClientUser, ou algo do tipo
O model não representa uma conta, o model representa um usuário do tipo cliente
| end | ||
|
|
||
|
|
||
| create_table "contract", force: :cascade do |t| |
There was a problem hiding this comment.
Precisamos conversar
Existe commit na main que adicionou um monte de coisa não revisada, fora de um PR e com vários erros.
| @@ -0,0 +1,5 @@ | |||
| class AddPhysicalClientToEnderecos < ActiveRecord::Migration[7.0] | |||
| def change | |||
| add_reference :enderecos, :physical_client, null: false, foreign_key: true | |||
There was a problem hiding this comment.
Como discutimos quinta passada, endereço vai ser polimórfico e pra ser dessa forma esta migração está incompleta e a proxima é desnecessaria
| @@ -0,0 +1,5 @@ | |||
| class AddCorporateClientToEndereco < ActiveRecord::Migration[7.0] | |||
| def change | |||
| add_reference :enderecos, :corporate_client, null: false, foreign_key: true | |||
There was a problem hiding this comment.
Ver o comentario anterior
| @@ -0,0 +1,12 @@ | |||
| class CreateClientUsers < ActiveRecord::Migration[7.0] | |||
There was a problem hiding this comment.
Se a migração do devise já adicionaria todas essas informações, esta migração é desnecessária.
4 participants
Relatório de Atualizações
Foi criada uma nova model
ClientAccountpara atender às necessidades do projeto. Devido a conflitos de rota resultantes de duplicidade de código, foram realizadas modificações nas rotas para garantir a funcionalidade do projeto. Além disso, foi incluída a nova gemomniauth, a qual é essencial para a funcionalidade de autenticação do sistema.Detalhes das Modificações
ClientAccount.omniauthpara suporte à autenticação.Se houver necessidade de mais informações ou esclarecimentos sobre as implementações, estou à disposição.😊😊