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

Fix (Royalty) transport ships #392

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

SokyranTheDragon
Copy link
Member

Currently, sending a transport ships in some circumstances (like summoning one through royal permits) would result in them disappearing with all pawns and items they carry when arriving at their destinations.

The issue was caused due to transport ships themselves not being registered in shared cross refs, which caused them to fail syncing them as IExposable.

Currently, sending a transport ships in some circumstances (like summoning one through royal permits) would result in them disappearing with all pawns and items they carry when arriving at their destinations.

The issue was caused due to transport ships themselves not being registered in shared cross refs, which caused them to fail syncing them as IExposable.
@SokyranTheDragon SokyranTheDragon added bug Something isn't working. 1.4 Fixes or bugs relating to 1.4 (Not Biotech). royalty Fixes or bugs relating to Royalty. fix Fixes for a bug or desync. and removed 1.4 Fixes or bugs relating to 1.4 (Not Biotech). bug Something isn't working. labels Sep 1, 2023
@Zetrith
Copy link
Member

Zetrith commented Sep 1, 2023

Is it this issue? #369

@SokyranTheDragon
Copy link
Member Author

Seems like it is, as the logs are from 1.4. There was a basically identical issue in 1.2-1.3, but the shuttles were handled differently back then so solution would have been different back then.

@Zetrith Zetrith merged commit dee277d into rwmt:master Oct 17, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes for a bug or desync. royalty Fixes or bugs relating to Royalty.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants