Repository navigation
Bug(1031629): Resolved event rendering issue while switching doctors - #41
Merged
Merged
Conversation
sivakumars3442
approved these changes
Jun 12, 2026
VengateshManiraj
approved these changes
Jun 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug description
Bug 1031629: Doctor Non-Working Hours Not Displayed on Selection Until View Refresh
Root cause
The issue where scheduler appointments were intermittently not rendered after selecting a specialist was caused by a race condition between layout reinitialization and event rendering, triggered by an unconditional full scheduler refresh during dynamic updates.
In the existing implementation, the refreshDataSource method updates multiple scheduler‑level properties in a single execution flow, including WorkDays, resource filtering, working hours, and the event data source. These updates implicitly trigger an internal layout recalculation in the scheduler.
To address the missing events, the initial fix introduced a delayed full refresh:
await Task.Delay(50);
await scheduleObj.RefreshAsync();
It works fine in local, but fails in tunnel and staging.
Solution description
Calling RefreshAsync(false) resolves the race condition - LightWeight rerender
Code Studio usage(Mandatory)
Code Studio used in this PR/MR?
If
Yes: Primary use (choose one)Outcome
If “Cost time” explain in short (1 or 2 lines):
Impact assessment
Reason for not identifying earlier
This issue was not identified earlier as it is dependent on asynchronous timing and environment‑specific rendering behavior. The initial fix appeared stable in local testing but failed under tunnel and staging conditions due to differences in render and communication latency.
Areas tested against this fix
Breaking changes
breaking-issue)If yes, provide breaking commit details link and migration guidance.
Regression testing
Action taken to prevent recurrence
Automation status
Cross-platform verification
Related issues
Is this issue present in EJ2 or other components?
needs-attention-coreteam)Output screenshots
Before:
https://github.com/user-attachments/assets/ed4c153b-d2df-49f1-bb22-47c7a7e837bc
After:
https://github.com/user-attachments/assets/d190cfae-1426-49ec-9b40-4e4304307b13
API changes
Performance verification
Reviewer Checklist