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

When the calculated field is updated, pubSub does not work #116

Open
OllegK opened this issue Oct 8, 2020 · 4 comments
Open

When the calculated field is updated, pubSub does not work #116

OllegK opened this issue Oct 8, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@OllegK
Copy link
Contributor

OllegK commented Oct 8, 2020

SWE_PROP_BC_NOTI_NEW_RECORD_DATA is not honoured

@OllegK OllegK added the enhancement New feature or request label Oct 8, 2020
@OllegK OllegK changed the title Calculated field is updated, pubSub does not work When the calculated field is updated, pubSub does not work Oct 8, 2020
@OllegK
Copy link
Contributor Author

OllegK commented Oct 28, 2020

It was reported that pubSub does not work when just a new value is picked from the pick applet, and the record is saved (AK) (maybe this issue could be reproduced only with the not immediate post change field? ). As far as I remember, the cp was skipped because it was received when the mvg/pick applet "opened" on the not-committed record.

AK workaround
image

@ostgals
Copy link
Contributor

ostgals commented Nov 4, 2020

Actually yesterday I faced an issue that Siebel can send several SWE_PROP_BC_NOTI_STATE_CHANGED notifications with 'cp' state set either to 0 and to 1 (within single notification session). As a result UI was updated like the record was already saved, while actually it wasn't. The issue raised in an applet with complicated series of calculation while creating new record, triggering its 'commit pending' state back and forth several times.
So I have reworked my workaround to trigger UI updating only once with the latest 'cp' value after SWE_PROP_BC_NOTI_END notification arrived.

@esneko
Copy link
Member

esneko commented Nov 4, 2020

@ostgals Please can you make a PR with the SWE_PROP_BC_NOTI_END workaround for the nexus-bridge, or does your implementation rely heavily on React?

@ostgals
Copy link
Contributor

ostgals commented Nov 4, 2020

@esneko Yes, currently we have implemented this workaround as custom React hook. But I'll try to find time and to create PR to nexus-bridge next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants