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.

Sign up for GitHub

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

Jump to bottom

MONIT - Quitar el atributo requerido en mail y telefono #55

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

MONIT - Quitar el atributo requerido en mail y telefono #55

wants to merge 1 commit into from

Conversation

leon-bravo
Copy link
Contributor

@leon-bravo leon-bravo commented Jul 1, 2022 •
edited
Loading

Requerimiento

MONIT-44 - Campos NO obligatorios - Monitoreo de activaciones

Funcionalidad desarrollada

  1. Se quito el atributo requerido en el campo telefono

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@leon-bravo leon-bravo requested review from a team as code owners July 1, 2022 12:30
@leon-bravo leon-bravo requested a review from martinebucarey July 8, 2022 11:58
@martinebucarey
Copy link
Contributor

USUARIO: lbravo
BUILD NUMBER: 7501
CYPRESS RUN: 6306
TEST START: 2022-08-23T16:20:02.401Z
TOTAL: 398
SUCCESS: 391
FAIL: 6
SKIPPED: 1

AgosLizzi
AgosLizzi requested changes Aug 25, 2022
View reviewed changes
Copy link

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Si el campo email se utiliza para administrar las cuentas y por tanto es "obligatorio" tenerlo, yo le agregaría como recurso visual el asterisco.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AgosLizzi AgosLizzi AgosLizzi requested changes

@plammel plammel Awaiting requested review from plammel plammel is a code owner automatically assigned from andes/code-review

@martinebucarey martinebucarey Awaiting requested review from martinebucarey

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
test fail
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@leon-bravo @martinebucarey @AgosLizzi