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: Empty mandatory fields. #699

Conversation

EdwinBetanc0urt
Copy link
Collaborator

Bug report / Feature

Steps to reproduce

1 - Open Employee (All) window.
2 - Press new option.
3 - Fill Search Key field.
4 - Change focus from a field to other field.

Screenshot or Gif

mandatory-empty-fields-error

Link to minimal reproduction

https://demo.erpya.com/#/7aa4242a-93c0-42d8-92be-8250002d3e3c/d97027fd-4cd5-445e-8fd8-ef5d3f7959b4/window/53111?action=create-new&tabParent=0&tabChild=0

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 #698

@EdwinBetanc0urt EdwinBetanc0urt self-assigned this Mar 30, 2021
@EdwinBetanc0urt EdwinBetanc0urt added 24 Basic Changes 02 Bug Something isn't working 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day labels Mar 30, 2021
@yamelsenih yamelsenih merged commit 2e39e18 into adempiere:develop Mar 30, 2021
yamelsenih added a commit to adempiere/adempiere-vue-docker that referenced this pull request Apr 4, 2021
…t-3.4)

Hi erveryone, here a new release
[rt-3.4](https://github.com/adempiere/adempiere-vue/releases/tag/rt-3.4)
with many changes. We are happy of share it with all:

## Related Commits

- fix: Empty mandatory fields.
[#699](adempiere/adempiere-vue#699)
- Bugfix/#692 context menu butons
[#700](adempiere/adempiere-vue#700)
- add buttom new record
[#703](adempiere/adempiere-vue#703)
- change style the button new order
[#704](adempiere/adempiere-vue#704)
- Remove unnecessary request utility
[#706](adempiere/adempiere-vue#706)
  - Use default request utility from legacy source code
  - Improve calling structure
  - Implement posibility of create endpoint for request
  - Allows define a generic calling for backend service
  - Remove unnecessary functions
- fix: Window references.
[#710](adempiere/adempiere-vue#710)
- fix: Displayed value current document status.
[#709](adempiere/adempiere-vue#709)
- feat: Add docker compose to deploy all stack.
[#711](adempiere/adempiere-vue#711)
- Improve request util
[#712](adempiere/adempiere-vue#712)
  - Improve request util
  - Improve request util based on basic and generic request
  - Add ADempiere request layer (use it as template for others)
  - Add Release information from github
 - Chamge ADempiere request API fos a custom request

## Available Docker Images:

- [ADempiere
gRPC](https://hub.docker.com/r/erpya/adempiere-grpc-all-in-one)
```shell
docker run -d \
    -it \
    --name adempiere-grpc-all-in-one \
    -p 50059:50059 \
    -e SERVER_PORT=50059 \
    -e SERVICES_ENABLED="access; business; core; dashboarding;
dictionary; enrollment; log; ui; workflow; store; pos; updater;" \
    -e SERVER_LOG_LEVEL="WARNING" \
    -e DB_HOST="localhost" \
    -e DB_PORT="5432" \
    -e DB_NAME=\"adempiere\" \
    -e DB_USER="adempiere" \
    -e DB_PASSWORD="adempiere" \
    -e DB_TYPE="PostgreSQL" \
    erpya/adempiere-grpc-all-in-one
```
- [Proxy ADempiere
API](https://hub.docker.com/r/erpya/proxy-adempiere-api)
```shell
docker run -it -d \
    --name proxy-adempiere-api \
    -p 8085:8085 \
    -e AD_DEFAULT_HOST="Your_gRPC_Server_IP" \
    -e AD_DEFAULT_PORT="50059" \
    -e AD_TOKEN="Your_ADempiere_Generated_Token" \
    -e VS_ENV="dev" \
    -e RESTORE_DB="N" \
    erpya/proxy-adempiere-api
```
- [ADempiere Vue](https://hub.docker.com/r/erpya/adempiere-vue)
```shell
docker run -it \
    --name adempiere-vue \
    -p 80:80 \
    -e API_URL="http://localhost:8085" \
    erpya/adempiere-vue
```
- [ADempiere](https://hub.docker.com/repository/docker/erpya/adempiere-base)
```
docker pull erpya/adempiere-base
docker run -p 8888:8888 erpya/adempiere-base
```

## Available Previews

- [ADempiere-Vue](https://demo-ui.erpya.com)
- [ADempiere-ZK](https://demo.erpya.com)

Authentication values:
### A Demo
```
User: demo
Password: demo
```
### Garden World
```
User: GardenAdmin
Password: GardenAdmin
```
@EdwinBetanc0urt EdwinBetanc0urt deleted the bugfix/#698-empty-field-mandatory branch May 27, 2021 03:31
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 24 Basic Changes 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug Report] columnName message after change focus
2 participants