Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Grid view not refreshed on websocket event #489

Closed
teosarca opened this issue Mar 4, 2017 · 3 comments
Closed

Grid view not refreshed on websocket event #489

teosarca opened this issue Mar 4, 2017 · 3 comments

Comments

@teosarca
Copy link
Member

teosarca commented Mar 4, 2017

Type of issue

Bug

Steps to reproduce

Test case:

  • Material receipt candidate window
  • select a row and receive an HU (via HU Editor) and create the material receipt for it
  • at that moment, the backend is sending a websocket event to ask the Material receipt candidate view to fresh

=> the frontend is not refreshing the view.

image

image

@teosarca teosarca added this to the 2017-10 milestone Mar 4, 2017
@damianprzygodzki damianprzygodzki self-assigned this Mar 6, 2017
@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented Mar 6, 2017

Can't reproduce. Tested like:

  • open material receipt candidates (540196, viewId: d541f7e3-1094-4c7b-a37a-06756acbe87b)
  • selected first row, default quick action (1 IFCO 6410 x 8 Stk) triggered, HU editor opened
  • selected first row, default quick action (Create mat. rec.) triggered, both views refresh by socket event
websocket refresh  2158f79e-f3f8-4839-8684-2cbb34cdd7f4 // this one inside modal
websocket refresh  d541f7e3-1094-4c7b-a37a-06756acbe87b // mat. rec. candidate

The same after reverse, both events occured, triggered fetching data for both document lists.

@teosarca
Copy link
Member Author

teosarca commented Mar 6, 2017

Hi @damianprzygodzki

I think the issue is reproductible only if:

I suspect that the frontend simply does not start listening to view's endpoint in that case.

If u cannot reproduce pls ping me ;)

damianprzygodzki added a commit that referenced this issue Mar 6, 2017
#489 ConnectWS moved to after data fetching
@metas-mk metas-mk self-assigned this Mar 6, 2017
@metas-mk
Copy link
Member

metas-mk commented Mar 6, 2017

Tested. Works nicely now.

metas-mk added a commit to metasfresh/metasfresh that referenced this issue Mar 6, 2017
Adding metasfresh/metasfresh-webui-frontend-legacy#489 Grid view not refreshed on websocket event to our next release.
@metas-mk metas-mk closed this as completed Mar 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants