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

Make trade analyzer module into a non-removable module #3474

Merged
merged 1 commit into from Oct 10, 2015
Merged

Make trade analyzer module into a non-removable module #3474

merged 1 commit into from Oct 10, 2015

Conversation

joonicks
Copy link
Contributor

@joonicks joonicks commented Oct 4, 2015

step 2 would be to alter the ship mechanic to actually being able to
purge it from the ship if the player so wished...

step 2 would be to alter the ship mechanic to actually being able to
purge it from the ship if the player so wished...
@joonicks
Copy link
Contributor Author

joonicks commented Oct 4, 2015

screenshot-20151004-132332

@impaktor
Copy link
Member

impaktor commented Oct 4, 2015

I just posted some general thoughts, slightly related to this PR:
http://pioneerspacesim.net/forum/viewtopic.php?f=3&t=364

@fluffyfreak
Copy link
Contributor

I have no strong feelings about this one way of the other but I do question whether something needs to be a piece of "equipment" if it cannot be removed and everyone always has one?

Isn't it then just a function of the machine and thus we should remove the check to see if you have one and just assume that you do?

@impaktor
Copy link
Member

impaktor commented Oct 8, 2015

I kind of feel this doesn't serve much function on its own. Maybe if there was more "features". As it stands, it feels like it's removing functionality.

I can think of the trade analyzer (Maybe it should have been called "trade computer" in the first place?) as beeing a physical cartridge, or expansion board to a computer. Maybe the player wants to sell it to get the cash?

@fluffyfreak
Copy link
Contributor

Ok now that I understand what this is for I think it makes sense especially if it's the foundations for future work. Merging.

fluffyfreak added a commit that referenced this pull request Oct 10, 2015
Make trade analyzer module into a non-removable module
@fluffyfreak fluffyfreak merged commit 7cb2481 into pioneerspacesim:master Oct 10, 2015
fluffyfreak added a commit that referenced this pull request Jun 26, 2016
Make trade analyzer removable again. Undo #3474, close #3742
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.

None yet

3 participants