Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-10-27 11:02:16 +01:00
Code Issues Projects Releases Wiki Activity

Merge pull request #805 from Unleash/fix-rollout-datatype

Browse Source 
fix: change rollout datatype to string
This commit is contained in:
sighphyre 2022-03-23 12:07:54 +02:00 committed by GitHub
commit 47e43dc23c
3 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
frontend/cypress/integration/feature/feature.spec.ts
View File

@ -114,7 +114,7 @@ describe('feature', () => {
expect(req.body.name).to.equal('flexibleRollout'); expect(req.body.name).to.equal('flexibleRollout');
expect(req.body.parameters.groupId).to.equal(featureToggleName); expect(req.body.parameters.groupId).to.equal(featureToggleName);
expect(req.body.parameters.stickiness).to.equal('default'); expect(req.body.parameters.stickiness).to.equal('default');
expect(req.body.parameters.rollout).to.equal(30); expect(req.body.parameters.rollout).to.equal('30');
if (enterprise) { if (enterprise) {
expect(req.body.constraints.length).to.equal(1); expect(req.body.constraints.length).to.equal(1);
@ -160,7 +160,7 @@ describe('feature', () => {
req => { req => {
expect(req.body.parameters.groupId).to.equal('new-group-id'); expect(req.body.parameters.groupId).to.equal('new-group-id');
expect(req.body.parameters.stickiness).to.equal('sessionId'); expect(req.body.parameters.stickiness).to.equal('sessionId');
expect(req.body.parameters.rollout).to.equal(60); expect(req.body.parameters.rollout).to.equal('60');
if (enterprise) { if (enterprise) {
expect(req.body.constraints.length).to.equal(1); expect(req.body.constraints.length).to.equal(1);

7
frontend/src/component/feature/StrategyTypes/FlexibleStrategy/FlexibleStrategy.tsx
View File

@ -44,7 +44,7 @@ const FlexibleStrategy = ({
e: React.ChangeEvent<{}>, e: React.ChangeEvent<{}>,
value: number | number[] value: number | number[]
) => { ) => {
updateParameter('rollout', value); updateParameter('rollout', value.toString());
}; };
const resolveStickiness = () => const resolveStickiness = () =>
@ -62,7 +62,8 @@ const FlexibleStrategy = ({
const stickinessOptions = resolveStickiness(); const stickinessOptions = resolveStickiness();
const rollout = parameters.rollout !== undefined ? parameters.rollout : 100; const rollout =
parameters.rollout !== undefined ? parameters.rollout : '100';
const stickiness = parameters.stickiness; const stickiness = parameters.stickiness;
const groupId = parameters.groupId; const groupId = parameters.groupId;
@ -70,7 +71,7 @@ const FlexibleStrategy = ({
<div> <div>
<RolloutSlider <RolloutSlider
name="Rollout" name="Rollout"
value={1 * rollout} value={parseInt(rollout)}
disabled={!editable} disabled={!editable}
onChange={updateRollout} onChange={updateRollout}
/> />

2
frontend/src/interfaces/strategy.ts
View File

@ -28,7 +28,7 @@ export interface IConstraint {
export interface IParameter { export interface IParameter {
groupId?: string; groupId?: string;
rollout?: number; rollout?: string;
stickiness?: string; stickiness?: string;
[index: string]: any; [index: string]: any;