-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Outfit wars support #314
Outfit wars support #314
Conversation
This should eventually do something different with the outfit-wars data so that it doesn't appear in the normal active alerts on the website
Updated CreateInstanceOutfitWarsDto and associated entity
Updated outfit wars instance entity to properly use correct index and other props
…alerts by default. Also added to entity. Fixed descriptions for ps2alertsEventType in OW entity
@@ -0,0 +1,147 @@ | |||
// import {Inject, Injectable, Logger} from '@nestjs/common'; |
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 file shouldn't exist in this PR, my bad
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.
Various notes
import {Column} from 'typeorm'; | ||
|
||
export default class OutfitwarsMapControlEmbed { | ||
@ApiProperty({example: 20, description: 'Team 1 (blue) percentage'}) |
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.
Old descriptions
@@ -43,5 +43,5 @@ export class UpdateFacilityControlDto { | |||
@IsObject() | |||
@IsOptional() | |||
@ApiModelProperty({example: null, default: null}) | |||
mapControl: MapControlEmbed; | |||
mapControl: TerritoryControlMapControlEmbed; |
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.
Thinking we should probably prefix this with metagame so it's clear is not OW related, like the rest are
@IsNotEmpty() | ||
@IsOptional() | ||
@ApiModelProperty({example: false, default: false}) | ||
isInitial: boolean; |
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.
Potentially can remove
@ApiOperation({summary: 'Returns a single outfit wars instance'}) | ||
@ApiResponse({ | ||
status: 200, | ||
description: 'The Metagame Instance', |
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.
Wonky description
return await this.mongoOperationsService.deleteOne(InstanceOutfitWarsTerritoryEntity, {instanceId}); | ||
} | ||
|
||
@Get('/active') |
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.
Needs testing
{ | ||
state: Ps2alertsEventState.STARTED, | ||
world, | ||
zone, |
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.
Zone may not be needed here
|
Also adds endpoint to update that instanceId, and converts falcon interface to strings
…e used for search in the future
Tided up various implicit queries
This adds the start time of the next match to the outfits' rankings, and makes the cron job non-destructive to instanceId and createdAt timestamp so it can be run more frequently
Work towards adding support for outfit wars. Closes #312
Tasks
ps2alertsEventType
ps2alertsEventType
migration to ALL aggregates, giving all current records the value of1
(live metagame)