-
-
Notifications
You must be signed in to change notification settings - Fork 376
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
Restrict entry to Rondel system #5095
Conversation
I like the direction, however, I wonder:
Those are my initial thoughts. I'd stress adding hit to Advice module, and some loophole for player to be granted access. Also, making it into the system, there should be some reward, like cheap slaves. I know slavery is illegal in Haber, but I'd suggest we change it, they're supposed to be "evil", so... |
local Lang = require 'Lang' | ||
local Game = require 'Game' | ||
local Space = require 'Space' | ||
local cargo = require 'Commodities' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks to be unused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I will take care of that. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I happen to be using that, actually.
In order, replying to @impaktor ;
|
Isn't 2 minutes a bit too short? Maybe 5 and 15? |
OK, I assumed there was an inhabited system. If there was, response time could be something with the distance between player and main planet, or similar. |
Having not looked at code yet (on mobile), I would recommend making this module completely independent of the actual system. I will look into adding a permit system to the Character framework which should make this easy. Regarding the gameplay implications, I'd recommend extending the time to five minutes for all drives and making sure that time acceleration doesn't adversely impact the experience (possibly stop time accel when receiving the message). Organization wise, I'd recommend adding this to the PolicePatrol module folder and associated language resource - there's no limitation that requires only one lua file per module. (And with the |
I have nothing to say about the code/lcoation/etc but I do like the idea. Perhaps in future this can be modified for any restricted system which could just be a flag on the system itself. For now I'd be happy to have it be specific to Rondel, then change it in when the time comes. |
Moved Rondel.lua to PolicePatrol directory; increased time to 10 mins for civilian drives, 5 mins for military drives. |
Added some rumour/backstory about Rondel to the advice module. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good and based on prior discussion I think this is getting ready to merge. I only have one change request, and that is to check for Rondel using a SystemPath instead of a name that can easily be duplicated in procgen.
I know the title makes it sound stupid (why would you stop the players from entering a system?), but hear me out. I noticed a red spot on the sector map and found this Haber system in the middle of SolFed territory.
It looks like someone made this system just as a custom system example (no starports, no defined planets etc.), but I thought it could use more than just a fancy description... some more excitement. Which brings me to this PR:
Upon entering the system, 7 police vehicles appear near the player and warn them to leave. They are quite paranoid too:
As for the code, it has been almost entirely copied from the PolicePatrol module, and edited.