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
Reworking of challenge system #50
Conversation
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
Signed-off-by: Reece Dunham <me@rdil.rocks>
@@ -811,51 +737,49 @@ export class ChallengeService extends ChallengeRegistry { | |||
gameVersion: GameVersion, | |||
userId: string, | |||
): CompiledChallengeTreeData { | |||
let contract | |||
let contract: MissionManifest | null | |||
|
|||
// TODO: Properly get escalation groups for this | |||
if (challenge.Type === "contract") { | |||
contract = this.controller.resolveContract( | |||
challenge.InclusionData?.ContractIds?.[0] || "", | |||
) | |||
|
|||
// This is so we can remove unused data and make it more like official - AF |
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.
This should be investigated as to see whether the format matches official, I have a strong suspicion it may not 100% match. (I remember half arsing the format as it was a mess)
EDIT: The line I selected to add this comment, was on my comment, not your change.
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.
Logic all LGTM, untested within the game.
Talked about in #45
Just opening for initial thoughts - haven't even tested it yet