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

fix(cdu): revert socket changes #6944

Merged
merged 2 commits into from
Mar 20, 2022
Merged

fix(cdu): revert socket changes #6944

merged 2 commits into from
Mar 20, 2022

Conversation

Saschl
Copy link
Contributor

@Saschl Saschl commented Mar 20, 2022

Fixes #[issue_no]

Summary of Changes

Screenshots (if necessary)

References

Additional context

Discord username (if different from GitHub):

Testing instructions

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, click on the bottom PR tab
  4. Click on the A32NX download link at the bottom of the page

@Benjozork Benjozork merged commit 3f4c520 into flybywiresim:master Mar 20, 2022
ErickSharp pushed a commit that referenced this pull request Apr 3, 2022
@Saschl @ErickSharp
fix(cdu): revert socket changes (#6944)
b7aeacd 
* Revert "fix: close old websocket before every connection attempt (#6937)"

This reverts commit 774dbcb.

* chore: decrease check connection interval
@2hwk 2hwk added this to the v0.8.0 milestone Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Benjozork Benjozork Benjozork approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.8.0
Development

Successfully merging this pull request may close these issues.
3 participants
@Saschl @Benjozork @2hwk