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

Add title labels #20

Merged
merged 10 commits into from
Mar 19, 2020
Merged

Add title labels #20

merged 10 commits into from
Mar 19, 2020

Conversation

matias179
Copy link

Tomando como base el cambio que agrego @olivertar para seleccionar desde panel los tipos de documento, le agregue para que el merchant pueda seleccionar tambien el texto que acompaña esa seleccion de campos.

Escenarios de testing manual

Backend

  1. admin login
  2. Stores > Configuration :: Sales > Checkout :: Customer Identification Document
  3. Seleccionar el texto que va a aparecer en shipping/billing
  4. Guardar cambios y limpiar Mage cache.
  5. Verificar que las opciones seleccionadas fueron guardadas correctamente

Frontend

  1. Agregar cualquier producto al cart
  2. Iniciar checkout
  3. Verificar que el titulo que sale para seleccionar el documento sea el que se cargo en el panel (tanto en billing como en shipping)

@barbanet barbanet requested a review from a team March 12, 2020 21:15
Block/Adminhtml/Order/View/CidFields.php Outdated Show resolved Hide resolved
Block/Adminhtml/Order/View/CidFields.php Outdated Show resolved Hide resolved
view/frontend/web/js/view/checkout/cid-billing-form.js Outdated Show resolved Hide resolved
view/frontend/web/js/view/checkout/cid-billing-form.js Outdated Show resolved Hide resolved
@matias179
Copy link
Author

Agregue los cambios que hizo @olivertar en su rama master asi no hay problemas al aprobar el PR, y en base a lo que hablamos con Oliverio en Slack agregue dos campos de texto mas para permitir que el merchant pueda poner los motivos por lo que solicita el documento en los pasos de billing o de shipping en caso que lo prefiera

@matias179
Copy link
Author

Agregue un cambio para evitar usar Registry (issue #17 )
Optimice un poco el codigo, usando viewModel en lugar de bloques, ahora los bloques no son necesarios asi que los borre tambien.

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.

4 participants