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

fix: Field isActive not edit with false value. #693

Merged

Conversation

EdwinBetanc0urt
Copy link
Collaborator

Bug report / Feature

Screenshot or Gif

is-readonly-is-active-fix

Link to minimal reproduction

https://demo-ui.erpya.com/#/567acca9-225c-44ea-9a6e-809ea01199b2/a3e5c878-fb40-11e8-a479-7a0060f0aa01/window/190?tabParent=0&action=8af3f668-0c15-4017-a298-dcdca3586220

Other relevant information

  • Your OS: Linux Mint 19.1 Cinnamon x64.
  • Web Browser: Mozilla Firefox 85.0.
  • Node.js version: 12.20.0.
  • adempiere-vue version: 4.3.1.

Additional context

fixes #691

@EdwinBetanc0urt EdwinBetanc0urt self-assigned this Mar 24, 2021
@EdwinBetanc0urt EdwinBetanc0urt added (MNC) Minor Change (UIBF) UI Basic Features Base features for develop 02 Bug Something isn't working 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day and removed (MNC) Minor Change labels Mar 24, 2021
@yamelsenih
Copy link
Member

yamelsenih commented Mar 24, 2021

  • First test with a Active record: Good
    Peek 2021-03-23 23-42
  • Second test with a Inactive record: Fail
    Peek 2021-03-23 23-42-1

The bug with Active flag is fixed but is introduced a new bug when a window is opened and it has a record as inactive

@yamelsenih yamelsenih merged commit 31e6b7b into adempiere:develop Mar 25, 2021
@EdwinBetanc0urt EdwinBetanc0urt deleted the bugfix/#691-is-active-fix branch May 27, 2021 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
02 Bug Something isn't working 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day (UIBF) UI Basic Features Base features for develop
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug Report] Disabled record
2 participants