Skip to content

Commit

Permalink
fix: adjust feature-variant-updated event
Browse files Browse the repository at this point in the history
  • Loading branch information
ivarconr committed Dec 17, 2021
1 parent 0927b1d commit 8d8d480
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 6 deletions.
2 changes: 2 additions & 0 deletions src/lib/addons/datadog-definition.ts
Expand Up @@ -12,6 +12,7 @@ import {
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_PROJECT_CHANGE,
FEATURE_VARIANTS_UPDATED,
} from '../types/events';
import { IAddonDefinition } from '../types/model';

Expand Down Expand Up @@ -54,6 +55,7 @@ const dataDogDefinition: IAddonDefinition = {
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_PROJECT_CHANGE,
FEATURE_VARIANTS_UPDATED,
],
tagTypes: [
{
Expand Down
3 changes: 3 additions & 0 deletions src/lib/addons/feature-event-formatter-md.ts
Expand Up @@ -13,6 +13,7 @@ import {
FEATURE_METADATA_UPDATED,
FEATURE_PROJECT_CHANGE,
IEvent,
FEATURE_VARIANTS_UPDATED,
} from '../types/events';

export interface FeatureEventFormatter {
Expand Down Expand Up @@ -108,6 +109,8 @@ export class FeatureEventFormatterMd implements FeatureEventFormatter {
return 'created';
case FEATURE_UPDATED:
return 'updated';
case FEATURE_VARIANTS_UPDATED:
return 'updated variants for';
default:
return type;
}
Expand Down
2 changes: 2 additions & 0 deletions src/lib/addons/slack-definition.ts
Expand Up @@ -12,6 +12,7 @@ import {
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_PROJECT_CHANGE,
FEATURE_VARIANTS_UPDATED,
} from '../types/events';
import { IAddonDefinition } from '../types/model';

Expand Down Expand Up @@ -71,6 +72,7 @@ const slackDefinition: IAddonDefinition = {
FEATURE_STRATEGY_UPDATE,
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_VARIANTS_UPDATED,
FEATURE_PROJECT_CHANGE,
],
tagTypes: [
Expand Down
2 changes: 2 additions & 0 deletions src/lib/addons/teams-definition.ts
Expand Up @@ -12,6 +12,7 @@ import {
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_PROJECT_CHANGE,
FEATURE_VARIANTS_UPDATED,
} from '../types/events';
import { IAddonDefinition } from '../types/model';

Expand Down Expand Up @@ -42,6 +43,7 @@ const teamsDefinition: IAddonDefinition = {
FEATURE_STRATEGY_UPDATE,
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_VARIANTS_UPDATED,
FEATURE_PROJECT_CHANGE,
],
};
Expand Down
2 changes: 2 additions & 0 deletions src/lib/addons/webhook-definition.ts
Expand Up @@ -4,6 +4,7 @@ import {
FEATURE_ENVIRONMENT_DISABLED,
FEATURE_ENVIRONMENT_ENABLED,
FEATURE_METADATA_UPDATED,
FEATURE_VARIANTS_UPDATED,
FEATURE_PROJECT_CHANGE,
FEATURE_REVIVED,
FEATURE_STALE_OFF,
Expand Down Expand Up @@ -70,6 +71,7 @@ const webhookDefinition: IAddonDefinition = {
FEATURE_STRATEGY_UPDATE,
FEATURE_STRATEGY_ADD,
FEATURE_METADATA_UPDATED,
FEATURE_VARIANTS_UPDATED,
FEATURE_PROJECT_CHANGE,
],
};
Expand Down
12 changes: 6 additions & 6 deletions src/lib/types/events.ts
Expand Up @@ -146,23 +146,23 @@ export class FeatureVariantEvent extends BaseEvent {

readonly featureName: string;

readonly data: {
oldVariants: IVariant[];
newVariants: IVariant[];
};
readonly data: { variants: IVariant[] };

readonly preData: { variants: IVariant[] };

constructor(p: {
project: string;
featureName: string;
createdBy: string;
tags: ITag[];
oldVariants: IVariant[];
newVariants: IVariant[];
oldVariants: IVariant[];
}) {
super(FEATURE_VARIANTS_UPDATED, p.createdBy, p.tags);
this.project = p.project;
this.featureName = p.featureName;
this.data = { oldVariants: p.oldVariants, newVariants: p.newVariants };
this.data = { variants: p.newVariants };
this.preData = { variants: p.oldVariants };
}
}

Expand Down

1 comment on commit 8d8d480

@vercel
Copy link

@vercel vercel bot commented on 8d8d480 Dec 17, 2021

Choose a reason for hiding this comment

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

Please sign in to comment.