[WIP] decision flow: update role

Author: fenekku

Diff for: invenio_communities/assets/semantic-ui/js/invenio_communities/api/membershipRequests/api.js

@@ -5,9 +5,11 @@
 // Invenio-communities is free software; you can redistribute it and/or modify it
 // under the terms of the MIT License; see LICENSE file for more details.
 
-import { CommunityLinksExtractor } from "../CommunityLinksExtractor";
 import { http } from "react-invenio-forms";
 
+import { CommunityLinksExtractor } from "../CommunityLinksExtractor";
+import { bulkMembersSerializer } from "../serializers";
+
 /**
  * API Client for community membership requests.
  *
@@ -21,6 +23,14 @@ export class CommunityMembershipRequestsApi {
   }
 
   requestMembership = async (payload) => {
+    // assigned rather than defiend for ease of passing as callback
     return await http.post(this.linksExtractor.url("membership_requests"), payload);
   };
+
+  updateRole = async (membershipRequest, role) => {
+    // assigned rather than defiend for ease of passing as callback
+    const memberSerialized = bulkMembersSerializer([membershipRequest]);
+    const payload = { members: memberSerialized, role: role };
+    return await http.put(this.linksExtractor.url("membership_requests"), payload);
+  };
 }

Diff for: invenio_communities/assets/semantic-ui/js/invenio_communities/members/membership_requests/MembershipRequestsResultItem.js

@@ -15,8 +15,9 @@ import React, { Component } from "react";
 import { Image } from "react-invenio-forms";
 import { Grid, Item, Table } from "semantic-ui-react";
 
+import { MembershipRequestsContext } from "../../api/membershipRequests/MembershipRequestsContextProvider";
 import { RoleDropdown } from "../components/dropdowns";
-import { formattedTime } from "../utils";
+import { buildRequest, formattedTime } from "../utils";
 
 export class MembershipRequestsResultItem extends Component {
   constructor(props) {
@@ -25,6 +26,8 @@ export class MembershipRequestsResultItem extends Component {
     this.state = { membershipRequest: result };
   }
 
+  static contextType = MembershipRequestsContext;
+
   update = (data, value) => {
     const { membershipRequest } = this.state;
     this.setState({ membershipRequest: { ...membershipRequest, ...{ role: value } } });
@@ -39,13 +42,23 @@ export class MembershipRequestsResultItem extends Component {
     } = this.props;
 
     const {
-      membershipRequest: { member, request },
+      membershipRequest: { member },
       membershipRequest,
     } = this.state;
-    // TODO: Decision flow
-    // const { api: membershipRequestsApi } = this.context;
-    const rolesCanAssignByType = rolesCanAssign[member.type];
-    const membershipRequestExpiration = formattedTime(request.expires_at);
+    console.log("********************");
+    console.log("membershipRequest");
+    console.dir(membershipRequest);
+    console.log("********************");
+
+    const request = buildRequest(membershipRequest, ["accept", "decline"]);
+    console.log("********************");
+    console.log("request");
+    console.dir(request);
+    console.log("********************");
+
+    const { api: membershipRequestsApi } = this.context;
+    const roles = rolesCanAssign[member.type];
+    const expiration = formattedTime(request.expires_at);
     return (
       <Table.Row className="community-member-item">
         <Table.Cell>
@@ -73,17 +86,16 @@ export class MembershipRequestsResultItem extends Component {
           <RequestStatus status={request.status} />
         </Table.Cell>
         <Table.Cell
-          aria-label={i18next.t("Expires") + " " + membershipRequestExpiration}
+          aria-label={i18next.t("Expires") + " " + expiration}
           data-label={i18next.t("Expires")}
         >
-          {membershipRequestExpiration}
+          {expiration}
         </Table.Cell>
         <Table.Cell data-label={i18next.t("Role")}>
           <RoleDropdown
-            roles={rolesCanAssignByType}
+            roles={roles}
+            action={membershipRequestsApi.updateRole}
             successCallback={this.update}
-            // TODO: Decision flow
-            // action={membershipRequestsApi.updateRole}
             disabled={!membershipRequest.permissions.can_update_role}
             currentValue={membershipRequest.role}
             resource={membershipRequest}

Diff for: invenio_communities/members/resources/resource.py

@@ -39,6 +39,7 @@ def create_url_rules(self):
             route(
                 "GET", routes["membership_requests"], self.search_membership_requests
             ),
+            route("PUT", routes["membership_requests"], self.update_membership_requests),
         ]
 
     @request_view_args
@@ -160,3 +161,14 @@ def search_membership_requests(self):
             search_preference=search_preference(),
         )
         return hits.to_dict(), 200
+
+    @request_view_args
+    @request_extra_args
+    @request_data
+    def update_membership_requests(self):
+        """Update membership request.
+
+        From the outside, a membership request is its own resource.
+        From the inside, it's just a member when it comes to update.
+        """
+        return self.update()