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

[ReadHandler] Review EnginedScheduled flag logic #28052

Open
lpbeliveau-silabs opened this issue Jul 19, 2023 · 2 comments
Open

[ReadHandler] Review EnginedScheduled flag logic #28052

lpbeliveau-silabs opened this issue Jul 19, 2023 · 2 comments

Comments

@lpbeliveau-silabs
Copy link
Contributor

lpbeliveau-silabs commented Jul 19, 2023 •
edited
Loading

Current flag used for synchronized report emissions needs either rework, rename, documentation or all of the above.
@lpbeliveau-silabs lpbeliveau-silabs added the icd Intermittently Connected Devices label Jul 19, 2023
@github-actions github-actions bot removed the icd Intermittently Connected Devices label Jul 19, 2023
@lpbeliveau-silabs lpbeliveau-silabs mentioned this issue Jul 19, 2023
Merged
@lpbeliveau-silabs
Copy link
Contributor Author

The behavior of the sync scheduler when that flag is set or cleared also needs to be reviewed.

@lpbeliveau-silabs
Copy link
Contributor Author

lpbeliveau-silabs commented Jul 19, 2023 •
edited
Loading

It seems more exhaustive testing is also require on the sync mechanism to ensure it functions as expected.

@lpbeliveau-silabs lpbeliveau-silabs mentioned this issue Jul 25, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lpbeliveau-silabs