flagsmith icon indicating copy to clipboard operation
flagsmith copied to clipboard

Adding a segment overide causes the UI to crash

Open azriel46d opened this issue 3 years ago • 3 comments

I've added via API several test flags, however when attempting to generata a segment override the UI crashes. The API requests happening before the crash where

Request

fetch(".../api/v1/projects/1/features/262/", {
 ... 
 "referrerPolicy": "same-origin",
  "body": "{\"id\":262,\"name\":\"fake.central.black.Geri\",\"type\":\"STANDARD\",\"default_enabled\":true,\"initial_value\":null,\"created_date\":\"2022-09-09T12:38:27.100705Z\",\"description\":null,\"tags\":[],\"is_archived\":false,\"owners\":[{\"id\":1,\"email\":\"email@test\",\"first_name\":\"John\",\"last_name\":\"Doe\"}],\"project\":\"1\"}",
  "method": "PUT",
  "mode": "cors",
  "credentials": "include"
}); 

Response

{"id":262,"name":"fake.central.black.Geri","type":"STANDARD","default_enabled":true,"initial_value":null,"created_date":"2022-09-09T12:38:27.100705Z","description":null,"tags":[],"multivariate_options":[],"is_archived":false,"owners":[{"id":1,"email":"email@tes","first_name":"John","last_name":"Doe"}]}

Request

fetch(".../api/v1/features/feature-segments/", {
  "body": "{\"feature\":262,\"segment\":2,\"environment\":1,\"priority\":0}",
  "method": "POST",
  "mode": "cors",
  "credentials": "include"
});

Response

{"id":4,"feature":262,"segment":2,"environment":1,"priority":0}

Request

fetch(".../api/v1/features/featurestates/", {
  "body": "{\"enabled\":true,\"feature\":262,\"environment\":1,\"feature_segment\":4,\"feature_state_value\":{\"type\":\"unicode\",\"boolean_value\":null,\"integer_value\":null,\"string_value\":\"\"}}",
  "method": "POST",
  "mode": "cors",
  "credentials": "include"
});

Response

{"id":1294,"feature_state_value":{"type":"unicode","string_value":"","integer_value":null,"boolean_value":null},"multivariate_feature_state_values":[],"uuid":"cb8a1feb-3630-48d6-955d-f0621278bca6","enabled":true,"created_at":"2022-09-09T12:45:14.699088Z","updated_at":"2022-09-09T12:45:14.699103Z","version":1,"live_from":"2022-09-09T12:45:14.698885Z","feature":262,"environment":1,"identity":null,"feature_segment":4,"change_request":null}

Request

fetch(".../api/v1/features/feature-segments/update-priorities/", {
  "body": "[{\"id\":4,\"priority\":0}]",
  "method": "POST",
  "mode": "cors",
  "credentials": "include"
});

Response

[{"id":4,"segment":2,"priority":0,"environment":1}]

Request

fetch(".../api/v1/features/featurestates/1294/", {
  "body": "{\"id\":1294,\"feature_state_value\":{\"type\":\"unicode\",\"boolean_value\":null,\"integer_value\":null,\"string_value\":\"\"},\"uuid\":\"cb8a1feb-3630-48d6-955d-f0621278bca6\",\"enabled\":true,\"created_at\":\"2022-09-09T12:45:14.699088Z\",\"updated_at\":\"2022-09-09T12:45:14.699103Z\",\"version\":1,\"live_from\":\"2022-09-09T12:45:14.698885Z\",\"feature\":262,\"environment\":1,\"identity\":null,\"feature_segment\":4,\"change_request\":null}",
  "method": "PUT",
  "mode": "cors",
  "credentials": "include"
});

Response

{"id":1294,"feature_state_value":{"type":"unicode","string_value":"","integer_value":null,"boolean_value":null},"multivariate_feature_state_values":[],"uuid":"cb8a1feb-3630-48d6-955d-f0621278bca6","enabled":true,"created_at":"2022-09-09T12:45:14.699088Z","updated_at":"2022-09-09T12:45:14.877691Z","version":1,"live_from":"2022-09-09T12:45:14.698885Z","feature":262,"environment":1,"identity":null,"feature_segment":4,"change_request":null}

image

image

azriel46d avatar Sep 09 '22 12:09 azriel46d

Hi - can you confirm what version of the platform you are running? Is this self hosted or our SaaS platform?

dabeeeenster avatar Sep 09 '22 14:09 dabeeeenster

This is the self hosted version using the docker image with the following digest 9c1f994cdbeb

azriel46d avatar Sep 12 '22 07:09 azriel46d

Understood - we will take a look. cc @gagantrivedi !

dabeeeenster avatar Sep 12 '22 08:09 dabeeeenster

Pretty sure this is fixed in main - please reopen if not.

dabeeeenster avatar Mar 07 '23 15:03 dabeeeenster