Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix issue where device connection check is not memoized #11772

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

bjester
Copy link
Member

@bjester bjester commented Jan 23, 2024

Summary

I just happened to notice that a bit of code was incorrect, within the device connection checker logic. It should be using a memoized function to ensure the frontend doesn't query for connection testing too often for a device.

Screenshot from 2024-01-23 11-08-11

References

https://learningequality.slack.com/archives/CB37UM23A/p1705105053832879?thread_ts=1705100642.965639&cid=CB37UM23A

Reviewer guidance

I ran Kolibri and tested that the device selection modal still worked and tested connections appropriately.


Testing checklist

  • Contributor has fully tested the PR manually
  • If there are any front-end changes, before/after screenshots are included
  • Critical user journeys are covered by Gherkin stories
  • Critical and brittle code paths are covered by unit tests

PR process

  • PR has the correct target branch and milestone
  • PR has 'needs review' or 'work-in-progress' label
  • If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
  • If this is an important user-facing change, PR or related issue has a 'changelog' label
  • If this includes an internal dependency change, a link to the diff is provided

Reviewer checklist

  • Automated test coverage is satisfactory
  • PR is fully functional
  • PR has been tested for accessibility regressions
  • External dependency files were updated if necessary (yarn and pip)
  • Documentation is updated
  • Contributor is in AUTHORS.md

Copy link
Member

@rtibbles rtibbles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No regressions observed - network connections appear to be checked not too frequently.

@rtibbles rtibbles merged commit 44a4b22 into learningequality:release-v0.16.x Jan 23, 2024
34 checks passed
@rtibbles rtibbles self-assigned this Jan 24, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants