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: not sending delta when next position values are missing #255

Merged
merged 5 commits into from
Nov 16, 2024

Conversation

panaaj
Copy link
Member

@panaaj panaaj commented Jun 7, 2024

Fix RMB, BWC and BWR sentence processing behaviour to send a delta when position field values are missing.

Current behaviour:
Whe position field values are missing, function does not return data and no delta is sent.
This means that clients are left unaware of a previously active destination being "de-activated" and will continue to show the destination as active.

FIx:
When position field values are missing, return data so that a delta is sent with nextPoint.position value = null.

Send delta with nextPoint.position value = null
@panaaj panaaj added the fix label Jun 7, 2024
@panaaj panaaj requested a review from tkurki June 7, 2024 07:28
@panaaj panaaj changed the title Fix: RMB does not send delta when position field values are missing. Fix: RMB not sending delta when position values are missing. Jun 7, 2024
test/RMB.js Show resolved Hide resolved
@tkurki
Copy link
Member

tkurki commented Nov 13, 2024

Now that I think of it: at least BWC BWR also map to nextPoint, they should also be updated.

@tkurki tkurki changed the title Fix: RMB not sending delta when position values are missing. Fix: not sending delta when next position values are missing Nov 16, 2024
@tkurki tkurki merged commit 3564e65 into master Nov 16, 2024
5 checks passed
@tkurki tkurki deleted the fix-rmb-null-position branch November 16, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants