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

Split mission transfer into client and server #2160

Merged
merged 1 commit into from
Nov 4, 2023

Conversation

julianoes
Copy link
Collaborator

This was weirdly mixed up in the same class before and required another thread in the server plugin due to locking problems.

Here I go again 😄

Currently on top of #2060.

@julianoes julianoes force-pushed the pr-split-mission-transfer branch from fdb8428 to 7e6fc79 Compare October 30, 2023 19:53
@julianoes julianoes force-pushed the pr-split-ftp branch 2 times, most recently from a890302 to d1ec125 Compare November 2, 2023 03:55
@julianoes julianoes force-pushed the pr-split-mission-transfer branch from 7e6fc79 to 4209e2d Compare November 2, 2023 04:07
Base automatically changed from pr-split-ftp to main November 2, 2023 23:05
@julianoes julianoes marked this pull request as ready for review November 2, 2023 23:05
@julianoes julianoes force-pushed the pr-split-mission-transfer branch from 4209e2d to 8ea5f9f Compare November 2, 2023 23:06
This was weirdly mixed up in the same class before and required another
thread in the server plugin due to locking problems.

Signed-off-by: Julian Oes <[email protected]>
@julianoes julianoes force-pushed the pr-split-mission-transfer branch from 8ea5f9f to eecaaa2 Compare November 3, 2023 01:49
Copy link
Collaborator

@JonasVautherin JonasVautherin left a comment

Choose a reason for hiding this comment

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

A long one again 🙈 😆.

If mission transfer still works, that's fine for me!

@julianoes julianoes merged commit d4f0ca3 into main Nov 4, 2023
26 checks passed
@julianoes julianoes deleted the pr-split-mission-transfer branch November 4, 2023 05:06
@julianoes julianoes mentioned this pull request Dec 27, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants