Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
L
l10n_mx_toponyms
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • l10n_mx
  • l10n_mx_toponyms
  • Merge Requests
  • !14

Merged
Opened 6 years ago by Salvador Martínez@chavamm.83Developer

Fix filtro de dominio para neighborhood_id

  • Overview 4
  • Commits 1
  • Changes 1
All threads resolved
  • Loading...

Este MR es para establecer de manera correcta el dominio para el campo neighborhood_id ya que, al momento de seleccionar el codigo postal cuando aun no se selecciono primero el estado, el unico parametro de dominio para neighborhood_id era el valor del state_id relacionado con dicho zip_id seleccionado. Esto, resulta confuso para el usuario, ya que se le presenta un listado de colonias que no tienen relacion con el Codigo Postal seleccionado.

Ejemplo usando el modulo en el Tag 1.0.2 o 1.0.3:

  1. Seleccionar Ventas >> Clientes y dar click en Crear
  2. Establecer el Nombre del cliente
  3. Establecer el codigo postal 47460
  4. Se muestra un lista de colonias del estado de Jalisco
  5. El valor esperado es un listado de las colonias pertenecientes al código Postal 47460
    • Colinas del Valle
    • Lomas del Valle

Con este MR se resuelve ese problema.

Edited 6 years ago by Salvador Martínez
Request to merge chavamm.83:fix-neighborhood into master
  • Download as
  • Email patches

  • Plain diff

Checking pipeline status.

Checking approval status

Merged by Cuauhtémoc Díaz Minor 6 years ago (Jun 15, 2018 10:14pm GMT+0000) 6 years ago

The changes were merged into master with fce79f8b

Could not retrieve the pipeline status. For troubleshooting steps, read the documentation.


  • Salvador Martínez @chavamm.83 changed the description 6 years ago

    changed the description

  • Salvador Martínez
    Salvador Martínez @chavamm.83 · 6 years ago
    Developer

    Buen dia!!

    Espero sus comentarios @pyerp

    Saludos!

  • Agustín Cruz
    Agustín Cruz @agb80 · 6 years ago
    Owner

    Gracias por la corrección 👍

    ping @OpenPymeMx

  • Agustín Cruz @agb80 assigned to @cuauhtemoc.diaz 6 years ago

    assigned to @cuauhtemoc.diaz

  • Agustín Cruz @agb80 added Bug label 6 years ago

    added Bug label

  • Cuauhtémoc Díaz Minor
    Cuauhtémoc Díaz Minor @cuauhtemoc.diaz started a thread on the diff 6 years ago
    Resolved by Cuauhtémoc Díaz Minor 6 years ago
  • Cuauhtémoc Díaz Minor @cuauhtemoc.diaz resolved all discussions 6 years ago

    resolved all discussions

  • Cuauhtémoc Díaz Minor @cuauhtemoc.diaz merged 6 years ago

    merged

  • Cuauhtémoc Díaz Minor @cuauhtemoc.diaz mentioned in issue pyerp/bench#41 (closed) 6 years ago

    mentioned in issue pyerp/bench#41 (closed)

  • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
Please register or sign in to reply
0 Assignees
Assign to
0 Reviewers
Request review from
None
Milestone
None
Assign milestone
None
Time tracking
No estimate or time spent
1
Labels
Bug
Assign labels
  • No matching results
  • Manage project labels
Lock merge request
Unlocked
participants
Reference: l10n_mx/l10n_mx_toponyms!14
Source branch: fix-neighborhood

    0 pending comments