Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bloquei botão quando existe dose cadastrada (#371) #399

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jamessom
Copy link

Bloqueia o botão de registro de nova dosa, caso o usuário já possua uma dose cadastrada

💻 SCREENSHOTS

https://imgur.com/a/uwUrrGR
image

:squirrel: TESTS

SCENARIO ONE

  1. Acessar o endpoint de paciente
  2. Não será possivel clicar no botão de Registrar dose, caso o paciente já tenha uma dose cadastrada
  3. Caso o usuário não tenha dose alguma cadastrada, será possível clicar no botão

Describe what the expected results are.

📦 DEPLOY

If necessary, this section should be used to describe the deploy process.

Copy link
Collaborator

@Mat-Bit Mat-Bit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obrigado pela contribuição @jamessom

Fiz uma sugestão ao seu PR para adaptar a solicitação de alteração com a utilização que está sendo feita do sistema atualmente.

Veja se faz sentido para ti e qualquer dúvidas fico a disposição.

Comment on lines +21 to +25
if patient.doses.any?
link_class = 'btn btn-success disabled'
end

link_to "Registrar dose", new_admin_appointment_path(patient_id: patient.id), class: link_class
Copy link
Collaborator

@Mat-Bit Mat-Bit Oct 25, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if patient.doses.any?
link_class = 'btn btn-success disabled'
end
link_to "Registrar dose", new_admin_appointment_path(patient_id: patient.id), class: link_class
button_text = 'Registrar dose'
if patient.doses.any?
link_class = 'btn btn-info'
button_text += ' de reforço'
end
link_to button_text, new_admin_appointment_path(patient_id: patient.id), class: link_class

Nesse momento o botão está útil para registrar corretamente uma dose se o paciente já possui uma dose registrada.

Por isso faço essa sugestão no seu PR para podermos alinhar com o uso atual do sistema, o que achas?

(Talvez precise redimensionar o tamanho do botão para o texto caso o paciente já possua uma dose)

pr_399

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants