X Tutup
Skip to content

Fix #5406: broken scoop mission#5407

Merged
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
azieba:master
Oct 18, 2022
Merged

Fix #5406: broken scoop mission#5407
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
azieba:master

Conversation

@azieba
Copy link
Contributor

@azieba azieba commented Oct 16, 2022

Fixes #5406

@azieba azieba changed the title Fix for #5406: broken scoop mission Fix #5406: broken scoop mission Oct 16, 2022
Copy link
Member

@sturnclaw sturnclaw left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

@sturnclaw sturnclaw merged commit 9f7681d into pioneerspacesim:master Oct 18, 2022
@impaktor
Copy link
Member

@azieba Thanks for contribution. If you want to be in AUTHORS.txt, let us know what name to put.

@azieba
Copy link
Contributor Author

azieba commented Oct 18, 2022

@azieba Thanks for contribution. If you want to be in AUTHORS.txt, let us know what name to put.

That would be very generous of you to add me to the Authors.txt after committing just 3 characters to the code :)
I do not mind if you use my real name.
BR,
Andrzej Zięba

@impaktor
Copy link
Member

Done.

We sometimes forget to add new contributors, but there's no lower code contribution limit for being in the authors file, anything above 0 lines. (Translations (and comments?) don't count, I think).

That would be very generous of you to add me to the Authors.txt after committing just 3 characters to the code

Perhaps I have an ulterior motive, in getting you to submit more pull requests...

@sturnclaw sturnclaw mentioned this pull request Nov 6, 2022
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scoop missions do not work properly because onCargoDestroyed event is not emitted when cargo is scooped.

3 participants

X Tutup