Fix #356: Redirect to groups list after create/update

Kinto · Jun 3, 2019 · f1d398f · f1d398f
1 parent 0325164
commit f1d398f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/sagas/bucket.js b/src/sagas/bucket.js
@@ -277,7 +277,7 @@ export function* createGroup(
       data: groupData,
       safe: true,
     });
-    yield put(redirectTo("group:attributes", { bid, gid }));
+    yield put(redirectTo("bucket:groups", { bid }));
     yield put(notifySuccess("Group created."));
   } catch (error) {
     yield put(notifyError("Couldn't create group.", error));
@@ -305,7 +305,7 @@ export function* updateGroup(
     if (data) {
       const updatedGroup = { ...data, id: gid, last_modified };
       yield call([bucket, bucket.updateGroup], updatedGroup, { safe: true });
-      yield put(redirectTo("group:attributes", { bid, gid }));
+      yield put(redirectTo("bucket:groups", { bid }));
       yield put(notifySuccess("Group attributes updated."));
     } else if (permissions) {
       yield call([bucket, bucket.updateGroup], loadedData, {

diff --git a/test/sagas/bucket_test.js b/test/sagas/bucket_test.js
@@ -896,9 +896,8 @@ describe("bucket sagas", () => {
       it("should update the route path", () => {
         expect(createGroup.next().value).eql(
           put(
-            redirectTo("group:attributes", {
+            redirectTo("bucket:groups", {
               bid: "bucket",
-              gid: "group",
             })
           )
         );
@@ -976,9 +975,8 @@ describe("bucket sagas", () => {
         it("should update the route path", () => {
           expect(updateGroup.next().value).eql(
             put(
-              redirectTo("group:attributes", {
+              redirectTo("bucket:groups", {
                 bid: "bucket",
-                gid: "group",
               })
             )
           );