Merge pull request #1027 from Shoaibdev7/fix/owner-only-bounty-actions

Restrict Bounty Actions to Owners Only
stakwork · Feb 7, 2025 · 1c43868 · 1c43868
2 parents 133e28b + 3249175
commit 1c43868
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 25 deletions.
diff --git a/src/people/widgetViews/summaries/wantedSummaries/CodingBounty.tsx b/src/people/widgetViews/summaries/wantedSummaries/CodingBounty.tsx
@@ -91,6 +91,7 @@ function MobileView(props: CodingBountiesProps) {
     assigneeHandlerOpen,
     setCreatorStep,
     awards,
+    owner_id,
     setExtrasPropertyAndSaveMultiple,
     handleAssigneeDetails,
     peopleList,
@@ -493,6 +494,8 @@ function MobileView(props: CodingBountiesProps) {
 
   const isAssigner = person?.owner_pubkey === uiStore._meInfo?.owner_pubkey;
 
+  const hasOwnership = owner_id === uiStore._meInfo?.owner_pubkey;
+
   const handleStatusUpdate = async (proofId: string, status: BountyReviewStatus) => {
     try {
       console.log(proofId, status);
@@ -744,7 +747,7 @@ function MobileView(props: CodingBountiesProps) {
           hasAccess &&
           userAssigned && (
             <>
-              {!bountyCompleted && (
+              {hasOwnership && !bountyCompleted && (
                 <Button
                   disabled={paymentLoading || payBountyDisable || !created}
                   iconSize={14}
@@ -763,28 +766,31 @@ function MobileView(props: CodingBountiesProps) {
                 />
               )}
 
-              <IconButton
-                width={'100%'}
-                height={48}
-                disabled={
-                  paymentLoading || payBountyDisable || pendingPaymentLoading || bountyPending
-                }
-                style={{
-                  bottom: '10px'
-                }}
-                text={'Pay Bounty'}
-                loading={saving === 'paid' || updatingPayment}
-                textStyle={{
-                  display: 'flex',
-                  justifyContent: 'center',
-                  fontFamily: 'Barlow',
-                  fontSize: '15px',
-                  marginLeft: '30px'
-                }}
-                hovercolor={color.button_secondary.hover}
-                shadowcolor={color.button_secondary.shadow}
-                onClick={confirmPaymentHandler}
-              />
+              {hasOwnership && (
+                <IconButton
+                  width={'100%'}
+                  height={48}
+                  disabled={
+                    paymentLoading || payBountyDisable || pendingPaymentLoading || bountyPending
+                  }
+                  style={{
+                    bottom: '10px'
+                  }}
+                  text={'Pay Bounty'}
+                  loading={saving === 'paid' || updatingPayment}
+                  textStyle={{
+                    display: 'flex',
+                    justifyContent: 'center',
+                    fontFamily: 'Barlow',
+                    fontSize: '15px',
+                    marginLeft: '30px'
+                  }}
+                  hovercolor={color.button_secondary.hover}
+                  shadowcolor={color.button_secondary.shadow}
+                  onClick={confirmPaymentHandler}
+                />
+              )}
+
               {isOpenProofModal && (
                 <CodingBountyProofModal
                   closeModal={() => setIsOpenProofModal(false)}

diff --git a/src/people/widgetViews/summaries/wantedSummaries/CodingMobile.tsx b/src/people/widgetViews/summaries/wantedSummaries/CodingMobile.tsx
@@ -189,7 +189,7 @@ export default function MobileView(props: CodingViewProps) {
       )}
       <div style={{ padding: 20, overflow: 'auto', height: 'calc(100% - 60px)' }}>
         <Pad>
-          {!bountyPaid && hasAccess && (
+          {!bountyPaid && hasAccess && isAssigner && (
             <div
               style={{
                 display: 'flex',
@@ -292,7 +292,7 @@ export default function MobileView(props: CodingViewProps) {
                     marginLeft: '12px'
                   }}
                 />
-                {hasAccess && !bountyPaid && !bountyPending && (
+                {hasAccess && !bountyPaid && !bountyPending && isAssigner && (
                   <div
                     data-testid="edit-btn"
                     className="AssigneeCloseButtonContainer"