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

Remove unnecessary request utility #706

Merged
merged 1 commit into from
Apr 1, 2021
Merged

Remove unnecessary request utility #706

merged 1 commit into from
Apr 1, 2021

Conversation

yamelsenih
Copy link
Member

@yamelsenih yamelsenih commented Apr 1, 2021 •
edited
Loading

  • 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

@yamelsenih
Remove unnecessary request utility
f8badea 
- 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
@yamelsenih yamelsenih self-assigned this Apr 1, 2021
@yamelsenih yamelsenih added the 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day label Apr 1, 2021
@yamelsenih yamelsenih merged commit 6f128a0 into adempiere:develop Apr 1, 2021
@yamelsenih yamelsenih deleted the feature/#SC-Remove-Unnecessary-request-utility branch April 1, 2021 07:23
yamelsenih added a commit to adempiere/adempiere-vue-docker that referenced this pull request Apr 4, 2021
@yamelsenih
## [rt-3.4](https://github.com/adempiere/adempiere-vue/releases/tag/r…
b83b553 
…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 pushed a commit to EdwinBetanc0urt/adempiere-vue that referenced this pull request Jan 12, 2023
@elsiosanchez
Fix: Full Grid Mode (adempiere#706)
1fa79e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yamelsenih yamelsenih

Labels
24 Basic Changes 27 Fast Resolution Very fast Resolution for a issue, can be completed along the day (CIO) Code Improvement or Optimization Improve source code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yamelsenih