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

WIP lore/worldbuilding/station lobby #5283

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

joonicks
Copy link
Contributor

brings customization to space stations

json files tied to individual stations to provide attributes to base further customizations on

techLevel can be overridden per station

screenshot-20210926-162507

Update SpaceStation.lua
@@ -1893,7 +1893,7 @@
},
"STATION_DOCKS": {
"description": "Information shown in the station lobby, for total number of docking pads",
"message": "Our total capacity is {total_docking_pads} docking pads."
"message": "Your ship is docked on pad {shipbay} out of {bays} docking pads."
Copy link
Contributor

@nozmajner nozmajner Sep 26, 2021

Choose a reason for hiding this comment

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

Nice idea! Is the "out of bays" part necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I dont know but see the diff, how many bays there are in the station was all there was before, I only added the shipbay

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, then I realized that you replace the line, not adding another about the pads. Sorry

end

local station_docks = string.interp(l.STATION_DOCKS, { total_docking_pads = string.format("%d", station.numDocks),})
local substrings = { shipbay = station:GetAssignedBayNumber(Game.player),
Copy link
Member

Choose a reason for hiding this comment

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

The bays go from 1 up so I think this should probably be:
shipbay = station:GetAssignedBayNumber(Game.player) + 1

"description" : "",
"message" : "Moscow Spaceflight museum is a must see! See how it all started, including an actual moon lander recovered from the moon!"
},
"SOL_EARTH_OLYMPUS" : {
Copy link
Member

Choose a reason for hiding this comment

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

SOL_MARS_OLYMPUS
👽

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sol_mars_cydonia you mean?

Copy link
Member

Choose a reason for hiding this comment

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

No, Cydonia is another station. The text (ski slope...) shows correct on Olympus Mons station but the token name looked a bit off, that's all.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just abbreviated it because its unique enough with the sol_mars

Copy link
Member

Choose a reason for hiding this comment

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

its unique enough with the sol_mars

Yes, it would have been, but you named it SOL_EARTH_OLYMPUS. I don't know if we're talking about the same thing here and I may have misunderstood everything.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oopsies. carbon copy much? yes. I though you were talking about cydonia because of the alien emoji

Copy link
Member

Choose a reason for hiding this comment

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

Ah, the Little Green Men team! You need to also fix the string in SCC0000000013-OlympusMons.json. ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks,, seems im not a my best today.

@@ -42,7 +42,7 @@ local max_flavours = 22
local startLocations = {
{['name']=lui.START_AT_MARS,
['desc']=lui.START_AT_MARS_DESC,
['location']=SystemPath.New(0,0,0,0,18),
['location']=SystemPath.New(0,0,0,0,22),
Copy link
Member

Choose a reason for hiding this comment

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

There's an instance of '0,0,0,0,18' in /src/main.cpp also. For starting from the command line.

@Web-eWorks
Copy link
Member

Just FYI @joonicks I'd recommend marking this PR as a draft until you're ready to pull it out of WIP and get a full review on its features - it makes it a little easier for us to manage which PRs are still in progress and which are awaiting review and merge.

@joonicks
Copy link
Contributor Author

joonicks commented Oct 3, 2021

@Web-eWorks If only I knew how to

@joonicks joonicks marked this pull request as draft October 3, 2021 14:06
@joonicks joonicks changed the title WIP lore/station lobby WIP lore/worldbuilding/station lobby Oct 3, 2021
Update SCC0000000013-OlympusMons.json
incremental changes
@impaktor impaktor force-pushed the master branch 3 times, most recently from 876b6fc to b5bc47c Compare July 6, 2022 13:11
@impaktor impaktor force-pushed the master branch 2 times, most recently from 90e4530 to 6a3a044 Compare January 26, 2023 21:29
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

4 participants