From d9ddc3251605922c64d9c9c3697808ff602e46b1 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Thu, 28 Apr 2022 12:33:10 +0100
Subject: [PATCH 1/8] Fetch and display badges in org profile modal

---
 components/profile-modal.js | 51 +++++++++++++++++++++++++++++--------
 components/svg-square.js    |  9 +++++++
 pages/organization.js       | 16 ++++++++----
 3 files changed, 61 insertions(+), 15 deletions(-)
 create mode 100644 components/svg-square.js

diff --git a/components/profile-modal.js b/components/profile-modal.js
index bb83481d..18ae1c02 100644
--- a/components/profile-modal.js
+++ b/components/profile-modal.js
@@ -3,6 +3,7 @@ import { isEmpty } from 'ramda'
 import theme from '../styles/theme'
 import Popup from 'reactjs-popup'
 import Button from './button'
+import SvgSquare from '../components/svg-square'
 
 function renderActions (actions) {
   return (
@@ -43,7 +44,32 @@ function renderActions (actions) {
   )
 }
 
-export default function ProfileModal ({ user, attributes, onClose, actions }) {
+function renderBadges (badges) {
+  if (!badges || badges.length === 0) {
+    return null
+  }
+
+  return (
+    <table>
+      {badges.map((b) => (
+        <tr>
+          <td>
+            <SvgSquare color={b.color} />
+          </td>
+          <td>{b.name}</td>
+        </tr>
+      ))}
+    </table>
+  )
+}
+
+export default function ProfileModal ({
+  user,
+  attributes,
+  badges,
+  onClose,
+  actions
+}) {
   actions = actions || []
   let profileContent = <dl>User does not have a profile</dl>
   if (!isEmpty(attributes)) {
@@ -87,13 +113,18 @@ export default function ProfileModal ({ user, attributes, onClose, actions }) {
   `}</style>
     </>
   }
-  return <article className='modal__details'>
-    { user.img ? <img src={user.img} /> : '' }
-    <h2 style={{ display: 'flex', 'justifyContent': 'space-between' }}>
-      <span>{user.name}</span>
-      {!isEmpty(actions) && renderActions(actions)}
-    </h2>
-    {profileContent}
-    <Button size='small' onClick={() => onClose()}>close</Button>
-  </article>
+  return (
+    <article className='modal__details'>
+      {user.img ? <img src={user.img} /> : ''}
+      <h2 style={{ display: 'flex', justifyContent: 'space-between' }}>
+        <span>{user.name}</span>
+        {!isEmpty(actions) && renderActions(actions)}
+      </h2>
+      {profileContent}
+      {renderBadges(badges)}
+      <Button size='small' onClick={() => onClose()}>
+        close
+      </Button>
+    </article>
+  )
 }
diff --git a/components/svg-square.js b/components/svg-square.js
new file mode 100644
index 00000000..0c33d9bc
--- /dev/null
+++ b/components/svg-square.js
@@ -0,0 +1,9 @@
+import React from 'react'
+
+export default function SvgSquare ({ color, size = 20 }) {
+  return (
+    <svg width={size} height={size}>
+      <rect width={size} height={size} style={{ fill: color }} />
+    </svg>
+  )
+}
diff --git a/pages/organization.js b/pages/organization.js
index ff68fa15..e9a58849 100644
--- a/pages/organization.js
+++ b/pages/organization.js
@@ -8,6 +8,7 @@ import SectionHeader from '../components/section-header'
 import Table from '../components/table'
 import theme from '../styles/theme'
 import AddMemberForm from '../components/add-member-form'
+import SvgSquare from '../components/svg-square'
 import Button from '../components/button'
 import Modal from 'react-modal'
 import ProfileModal from '../components/profile-modal'
@@ -74,10 +75,18 @@ export default class Organization extends Component {
     const { id } = this.props
 
     try {
+      // Fetch profile attributes
       const profileInfo = await getUserOrgProfile(id, user.id)
+
+      // Fetch badges for this organization
+      const profileBadges = (
+        await apiClient.get(`/user/${user.id}/badges`)
+      ).badges.filter((b) => b.organization_id === parseInt(id))
+
       this.setState({
         profileInfo,
         profileMeta: user,
+        profileBadges,
         modalIsOpen: true
       })
     } catch (e) {
@@ -233,11 +242,7 @@ export default class Organization extends Component {
           <Table rows={(this.state.badges || []).map((row) => {
             return {
               ...row,
-              color: () => (
-                <svg width='20' height='20'>
-                  <rect width='20' height='20' style={{ fill: row.color }} />
-                </svg>
-              )
+              color: () => <SvgSquare color={row.color} />
             }
           })} columns={columns} onRowClick={
             ({ id: badgeId }) => Router.push(
@@ -415,6 +420,7 @@ export default class Organization extends Component {
         }} isOpen={this.state.modalIsOpen}>
           <ProfileModal
             user={this.state.profileMeta}
+            badges={this.state.profileBadges}
             attributes={this.state.profileInfo}
             onClose={this.closeProfileModal}
             actions={profileActions}

From 4067a0cd18ad7fee5deae05a2399cad9530a3924 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 10:58:08 +0100
Subject: [PATCH 2/8] Disable badge assignment by osm id

---
 pages/badges/edit.js | 38 +-------------------------------------
 1 file changed, 1 insertion(+), 37 deletions(-)

diff --git a/pages/badges/edit.js b/pages/badges/edit.js
index 917b2ceb..73970f01 100644
--- a/pages/badges/edit.js
+++ b/pages/badges/edit.js
@@ -94,7 +94,7 @@ export default class EditBadge extends Component {
       { key: 'validUntil', label: 'Valid Until' }
     ]
 
-    const { badge, assignablePeople } = this.state
+    const { badge } = this.state
     const users = (badge && badge.users) || []
 
     return (
@@ -102,42 +102,6 @@ export default class EditBadge extends Component {
         <div className='team__table'>
           <div className='page__heading'>
             <h2>Assigned Members</h2>
-            <Formik
-              initialValues={{ osmIdentifier: '' }}
-              onSubmit={async ({ osmIdentifier }) => {
-                const user = assignablePeople.find(
-                  (p) =>
-                    p.id === osmIdentifier ||
-                    p.name.toLowerCase() === osmIdentifier.toLowerCase()
-                )
-                if (!user) {
-                  toast.error('User is not part of this organization.')
-                } else {
-                  Router.push(
-                    join(
-                      URL,
-                      `/organizations/${orgId}/badges/${badgeId}/assign/${user.id}`
-                    )
-                  )
-                }
-              }}
-              render={({ values }) => {
-                return (
-                  <Form className='form-control'>
-                    <Field
-                      type='text'
-                      name='osmIdentifier'
-                      id='osmIdentifier'
-                      placeholder='OSM id'
-                      value={values.osmIdentifier}
-                    />
-                    <Button type='submit' variant='submit'>
-                      Assign
-                    </Button>
-                  </Form>
-                )
-              }}
-            />
           </div>
         </div>
 

From 75027dea07d100884ee8b3599ba18ed3ddcc0943 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 11:00:56 +0100
Subject: [PATCH 3/8] Display empty table message when a badge has no members
 assigned.

---
 pages/badges/edit.js | 33 ++++++++++++++++-----------------
 1 file changed, 16 insertions(+), 17 deletions(-)

diff --git a/pages/badges/edit.js b/pages/badges/edit.js
index 73970f01..6a705ee0 100644
--- a/pages/badges/edit.js
+++ b/pages/badges/edit.js
@@ -105,24 +105,23 @@ export default class EditBadge extends Component {
           </div>
         </div>
 
-        {users.length > 0 && (
-          <Table
-            rows={users.map((u) => ({
-              ...u,
-              assignedAt: u.assignedAt && toDateString(u.assignedAt),
-              validUntil: u.validUntil && toDateString(u.validUntil)
-            }))}
-            columns={columns}
-            onRowClick={({ id }) =>
-              Router.push(
-                join(
-                  URL,
-                  `/organizations/${orgId}/badges/${badgeId}/assign/${id}`
-                )
+        <Table
+          rows={users.map((u) => ({
+            ...u,
+            assignedAt: u.assignedAt && toDateString(u.assignedAt),
+            validUntil: u.validUntil && toDateString(u.validUntil)
+          }))}
+          emptyPlaceHolder='No members have this badge assigned. Badges can be assigned via user profile actions.'
+          columns={columns}
+          onRowClick={({ id }) =>
+            Router.push(
+              join(
+                URL,
+                `/organizations/${orgId}/badges/${badgeId}/assign/${id}`
               )
-            }
-          />
-        )}
+            )
+          }
+        />
       </section>
     )
   }

From 0eb74fbe375680edeef657982a547d6db2956c31 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 13:06:19 +0100
Subject: [PATCH 4/8] Create separated pages for badge assignments actions
 (new/edit)

---
 app/manage/index.js                           |  14 +-
 .../assign.js => badges-assignment/edit.js}   | 105 ++++-----
 pages/badges-assignment/new.js                | 213 ++++++++++++++++++
 3 files changed, 274 insertions(+), 58 deletions(-)
 rename pages/{badges/assign.js => badges-assignment/edit.js} (71%)
 create mode 100644 pages/badges-assignment/new.js

diff --git a/app/manage/index.js b/app/manage/index.js
index a44ca0be..4fd7b603 100644
--- a/app/manage/index.js
+++ b/app/manage/index.js
@@ -302,18 +302,18 @@ function manageRouter (nextApp) {
     }
   )
 
-  // Use same page for two routes
-  const assignBadgePageRoute = [
-    can('organization:edit'),
-    (req, res) => nextApp.render(req, res, '/badges/assign', req.params)
-  ]
+  // New badge assignment
   router.get(
     '/organizations/:id/badges/assign/:userId',
-    ...assignBadgePageRoute
+    can('organization:edit'),
+    (req, res) => nextApp.render(req, res, '/badges-assignment/new', req.params)
   )
+
+  // Edit badge assignment
   router.get(
     '/organizations/:id/badges/:badgeId/assign/:userId',
-    ...assignBadgePageRoute
+    can('organization:edit'),
+    (req, res) => nextApp.render(req, res, '/badges-assignment/edit', req.params)
   )
 
   return router
diff --git a/pages/badges/assign.js b/pages/badges-assignment/edit.js
similarity index 71%
rename from pages/badges/assign.js
rename to pages/badges-assignment/edit.js
index 9298cb7b..f65e2b4c 100644
--- a/pages/badges/assign.js
+++ b/pages/badges-assignment/edit.js
@@ -1,4 +1,5 @@
 import React, { Component } from 'react'
+import * as Yup from 'yup'
 import { Formik, Field, Form } from 'formik'
 import APIClient from '../../lib/api-client'
 import Button from '../../components/button'
@@ -7,6 +8,7 @@ import { toast } from 'react-toastify'
 import join from 'url-join'
 import Router from 'next/router'
 import getConfig from 'next/config'
+import { id } from 'date-fns/locale'
 
 const { publicRuntimeConfig } = getConfig()
 const URL = publicRuntimeConfig.APP_URL
@@ -39,13 +41,13 @@ function Section ({ children }) {
   )
 }
 
-export default class AssignBadge extends Component {
+export default class EditBadgeAssignment extends Component {
   static async getInitialProps ({ query }) {
     if (query) {
       return {
         orgId: query.id,
-        badgeId: query.badgeId,
-        userId: query.userId
+        badgeId: parseInt(query.badgeId),
+        userId: parseInt(query.userId)
       }
     }
   }
@@ -64,22 +66,26 @@ export default class AssignBadge extends Component {
   }
 
   async loadData () {
-    const { orgId, badgeId } = this.props
+    const { orgId, badgeId, userId } = this.props
 
     try {
       const org = await apiClient.get(`/organizations/${orgId}`)
-      let badge, badges
+      const badge = await apiClient.get(
+        `/organizations/${orgId}/badges/${badgeId}`
+      )
+      let assignment
+      if (badge && badge.users) {
+        assignment = badge.users.find((u) => u.id === parseInt(userId))
+      }
 
-      if (badgeId) {
-        badge = await apiClient.get(`/organizations/${orgId}/badges/${badgeId}`)
-      } else {
-        badges = await apiClient.get(`/organizations/${orgId}/badges`)
+      if (!assignment) {
+        throw Error('Badge assignment not found.')
       }
 
       this.setState({
         org,
         badge,
-        badges
+        assignment
       })
     } catch (error) {
       console.error(error)
@@ -99,8 +105,8 @@ export default class AssignBadge extends Component {
       return <div>Loading...</div>
     }
 
-    const { orgId, userId } = this.props
-    const { badges, badge, user } = this.state
+    const { orgId, userId, badgeId } = this.props
+    const { badge, assignment } = this.state
 
     return (
       <>
@@ -111,66 +117,57 @@ export default class AssignBadge extends Component {
           <Formik
             initialValues={{
               assignedAt:
-                (user && user.assignedAt && user.assignedAt.substring(0, 10)) ||
+                (assignment &&
+                  assignment.assignedAt &&
+                  assignment.assignedAt.substring(0, 10)) ||
                 format(Date.now(), 'yyyy-MM-dd'),
               validUntil:
-                (user && user.validUntil && user.validUntil.substring(0, 10)) ||
+                (assignment &&
+                  assignment.validUntil &&
+                  assignment.validUntil.substring(0, 10)) ||
                 ''
             }}
-            onSubmit={async ({ assignedAt, validUntil, badgeId }) => {
+            validationSchema={Yup.object().shape({
+              assignedAt: Yup.date().required(
+                'Please select an assignment date.'
+              ),
+              validUntil: Yup.date().when(
+                'assignedAt',
+                (assignedAt, schema) =>
+                  assignedAt &&
+                  schema.min(
+                    assignedAt,
+                    'End date must be after the start date.'
+                  )
+              )
+            })}
+            onSubmit={async ({ assignedAt, validUntil }) => {
               try {
                 const payload = {
                   assigned_at: assignedAt,
                   valid_until: validUntil !== '' ? validUntil : null
                 }
 
-                if (!user) {
-                  await apiClient.post(
-                    `/organizations/${orgId}/badges/${badgeId}/assign/${userId}`,
-                    payload
-                  )
-                  Router.push(
-                    join(
-                      URL,
-                      `/organizations/${orgId}/badges/${badgeId}/assign/${userId}`
-                    )
-                  )
-                } else {
-                  await apiClient.patch(
-                    `/organizations/${orgId}/member/${userId}/badge/${badgeId}`,
-                    payload
-                  )
-                  toast.info('Badge updated successfully.')
-                }
+                await apiClient.patch(
+                  `/organizations/${orgId}/member/${userId}/badge/${badgeId}`,
+                  payload
+                )
+                toast.info('Badge updated successfully.')
                 this.loadData()
               } catch (error) {
                 console.log(error)
                 toast.error(`Unexpected error, please try again later.`)
               }
             }}
-            render={({ isSubmitting, values }) => {
+            render={({ isSubmitting, values, errors, touched }) => {
               return (
                 <Form>
                   <div className='page__heading'>
                     <h2>User: {userId} (OSM id)</h2>
                   </div>
-                  {badge ? (
-                    <div className='page__heading'>
-                      <h2>Badge: {badge && badge.name}</h2>
-                    </div>
-                  ) : (
-                    <div className='form-control form-control__vertical'>
-                      <label htmlFor='badgeId'>Badge:</label>
-                      <Field as='select' name='badgeId'>
-                        <option value=''>Select a badge</option>
-                        {badges.map((b) => (
-                          <option key={b.id} value={b.id}>
-                            {b.name}
-                          </option>
-                        ))}
-                      </Field>
-                    </div>
-                  )}
+                  <div className='page__heading'>
+                    <h2>Badge: {badge && badge.name}</h2>
+                  </div>
                   <div className='form-control form-control__vertical'>
                     <label htmlFor='assignedAt'>Assigned At (required)</label>
                     <Field
@@ -178,6 +175,9 @@ export default class AssignBadge extends Component {
                       type='date'
                       value={values.assignedAt}
                     />
+                    {errors.assignedAt && (
+                      <div className='form--error'>{errors.assignedAt}</div>
+                    )}
                   </div>
                   <div className='form-control form-control__vertical'>
                     <label htmlFor='validUntil'>Valid Until</label>
@@ -186,6 +186,9 @@ export default class AssignBadge extends Component {
                       type='date'
                       value={values.validUntil}
                     />
+                    {errors.validUntil && (
+                      <div className='form--error'>{errors.validUntil}</div>
+                    )}
                   </div>
                   <ButtonWrapper>
                     <Button
diff --git a/pages/badges-assignment/new.js b/pages/badges-assignment/new.js
new file mode 100644
index 00000000..3441ccaf
--- /dev/null
+++ b/pages/badges-assignment/new.js
@@ -0,0 +1,213 @@
+import React, { Component } from 'react'
+import * as Yup from 'yup'
+import { Formik, Field, Form } from 'formik'
+import APIClient from '../../lib/api-client'
+import Button from '../../components/button'
+import { format } from 'date-fns'
+import { toast } from 'react-toastify'
+import join from 'url-join'
+import Router from 'next/router'
+import getConfig from 'next/config'
+
+const { publicRuntimeConfig } = getConfig()
+const URL = publicRuntimeConfig.APP_URL
+
+const apiClient = new APIClient()
+
+function ButtonWrapper ({ children }) {
+  return (
+    <div>
+      {children}
+      <style jsx global>{`
+      .button {
+        margin-right: 10px;
+      }
+    }`}</style>
+    </div>
+  )
+}
+
+function Section ({ children }) {
+  return (
+    <section>
+      {children}
+      <style jsx global>{`
+      section {
+        margin-bottom: 20px;
+      }
+    }`}</style>
+    </section>
+  )
+}
+
+export default class NewBadgeAssignment extends Component {
+  static async getInitialProps ({ query }) {
+    if (query) {
+      return {
+        orgId: query.id,
+        userId: query.userId
+      }
+    }
+  }
+
+  constructor (props) {
+    super(props)
+    this.state = {
+      badges: []
+    }
+
+    this.loadData = this.loadData.bind(this)
+  }
+
+  async componentDidMount () {
+    this.loadData()
+  }
+
+  async loadData () {
+    const { orgId } = this.props
+
+    try {
+      // Fetch data and apply to state
+      const org = await apiClient.get(`/organizations/${orgId}`)
+      const badges = await apiClient.get(`/organizations/${orgId}/badges`)
+      this.setState({
+        org,
+        badges
+      })
+    } catch (error) {
+      console.error(error)
+      this.setState({
+        error,
+        loading: false
+      })
+    }
+  }
+
+  renderPageInner () {
+    if (this.state.error) {
+      return <div>An unexpected error occurred, please try again later.</div>
+    }
+
+    if (!this.state.org && (!this.state.badge || !this.state.badges)) {
+      return <div>Loading...</div>
+    }
+
+    const { orgId, userId } = this.props
+    const { badges } = this.state
+
+    return (
+      <>
+        <div className='page__heading'>
+          <h1>Badge Assignment</h1>
+        </div>
+        <Section>
+          <Formik
+            initialValues={{
+              assignedAt: format(Date.now(), 'yyyy-MM-dd')
+            }}
+            validationSchema={Yup.object().shape({
+              badgeId: Yup.number()
+                .oneOf(badges.map((b) => b.id))
+                .required('Please select a badge.'),
+              assignedAt: Yup.date().required(
+                'Please select an assignment date.'
+              ),
+              validUntil: Yup.date().when(
+                'assignedAt',
+                (assignedAt, schema) =>
+                  assignedAt &&
+                  schema.min(
+                    assignedAt,
+                    'End date must be after the start date.'
+                  )
+              )
+            })}
+            onSubmit={async ({ assignedAt, validUntil, badgeId }) => {
+              try {
+                await apiClient.post(
+                  `/organizations/${orgId}/badges/${badgeId}/assign/${userId}`,
+                  {
+                    assigned_at: assignedAt,
+                    valid_until: validUntil
+                  }
+                )
+                Router.push(
+                  join(
+                    URL,
+                    `/organizations/${orgId}/badges/${badgeId}/assign/${userId}`
+                  )
+                )
+              } catch (error) {
+                console.log(error)
+                toast.error(`Unexpected error, please try again later.`)
+              }
+            }}
+            render={({ isSubmitting, values, errors, touched }) => {
+              return (
+                <Form>
+                  <div className='page__heading'>
+                    <h2>User: {userId} (OSM id)</h2>
+                  </div>
+
+                  <div className='form-control form-control__vertical'>
+                    <label htmlFor='badgeId'>Badge:</label>
+                    <Field as='select' name='badgeId'>
+                      <option value=''>Select a badge</option>
+                      {badges.map((b) => (
+                        <option key={b.id} value={b.id}>
+                          {b.name}
+                        </option>
+                      ))}
+                    </Field>
+                    {errors.badgeId && (
+                      <div className='form--error'>{errors.badgeId}</div>
+                    )}
+                  </div>
+                  <div className='form-control form-control__vertical'>
+                    <label htmlFor='assignedAt'>Assigned At (required)</label>
+                    <Field
+                      name='assignedAt'
+                      type='date'
+                      value={values.assignedAt}
+                    />
+                    {errors.assignedAt && (
+                      <div className='form--error'>{errors.assignedAt}</div>
+                    )}
+                  </div>
+                  <div className='form-control form-control__vertical'>
+                    <label htmlFor='validUntil'>Valid Until</label>
+                    <Field
+                      name='validUntil'
+                      type='date'
+                      value={values.validUntil}
+                    />
+                    {errors.validUntil && (
+                      <div className='form--error'>{errors.validUntil}</div>
+                    )}
+                  </div>
+                  <ButtonWrapper>
+                    <Button
+                      disabled={isSubmitting}
+                      variant='primary'
+                      type='submit'
+                      value='Assign'
+                    />
+                    <Button
+                      variant='small'
+                      href={`/organizations/${orgId}`}
+                      value='Go to organization view'
+                    />
+                  </ButtonWrapper>
+                </Form>
+              )
+            }}
+          />
+        </Section>
+      </>
+    )
+  }
+
+  render () {
+    return <article className='inner page'>{this.renderPageInner()}</article>
+  }
+}

From 48676e2a53e6823094325ff8b0e0f68291048a76 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 13:19:37 +0100
Subject: [PATCH 5/8] lint

---
 pages/badges-assignment/edit.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/pages/badges-assignment/edit.js b/pages/badges-assignment/edit.js
index f65e2b4c..f78c4a39 100644
--- a/pages/badges-assignment/edit.js
+++ b/pages/badges-assignment/edit.js
@@ -8,7 +8,6 @@ import { toast } from 'react-toastify'
 import join from 'url-join'
 import Router from 'next/router'
 import getConfig from 'next/config'
-import { id } from 'date-fns/locale'
 
 const { publicRuntimeConfig } = getConfig()
 const URL = publicRuntimeConfig.APP_URL
@@ -159,7 +158,7 @@ export default class EditBadgeAssignment extends Component {
                 toast.error(`Unexpected error, please try again later.`)
               }
             }}
-            render={({ isSubmitting, values, errors, touched }) => {
+            render={({ isSubmitting, values, errors }) => {
               return (
                 <Form>
                   <div className='page__heading'>

From 85ad8eb255c68c2cdd795eeeeffaece6d0ec7f17 Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 13:20:00 +0100
Subject: [PATCH 6/8] Enforce server timezone to UTC

---
 app/index.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/app/index.js b/app/index.js
index 0a1bfdb3..2afaf13c 100644
--- a/app/index.js
+++ b/app/index.js
@@ -1,3 +1,6 @@
+// Set server timezone to UTC to avoid issues with date parsing
+process.env.TZ = 'UTC'
+
 const path = require('path')
 const express = require('express')
 const bodyParser = require('body-parser')

From 04551c910d542c568a8e1146e826974e01a5768e Mon Sep 17 00:00:00 2001
From: Vitor George <vitor.george@gmail.com>
Date: Fri, 29 Apr 2022 13:25:44 +0100
Subject: [PATCH 7/8] Display better error message when the user already has
 the badge

---
 pages/badges-assignment/new.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/pages/badges-assignment/new.js b/pages/badges-assignment/new.js
index 3441ccaf..e083c673 100644
--- a/pages/badges-assignment/new.js
+++ b/pages/badges-assignment/new.js
@@ -139,7 +139,12 @@ export default class NewBadgeAssignment extends Component {
                 )
               } catch (error) {
                 console.log(error)
-                toast.error(`Unexpected error, please try again later.`)
+
+                if (error.message === 'User is already assigned to badge.') {
+                  toast.error(`User is already assigned to this badge, please select a different one.`)
+                } else {
+                  toast.error(`Unexpected error, please try again later.`)
+                }
               }
             }}
             render={({ isSubmitting, values, errors, touched }) => {

From b7cf630860d544d66af127c88d9390efa23602a5 Mon Sep 17 00:00:00 2001
From: Marc Farra <marcfarra@gmail.com>
Date: Fri, 29 Apr 2022 18:12:27 +0300
Subject: [PATCH 8/8] Fix conditional

---
 pages/organization.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/pages/organization.js b/pages/organization.js
index e9a58849..d45659ee 100644
--- a/pages/organization.js
+++ b/pages/organization.js
@@ -312,6 +312,7 @@ export default class Organization extends Component {
     let profileActions = []
 
     if (this.state.modalIsOpen && isUserOwner) {
+      console.log('here')
       const profileId = parseInt(this.state.profileMeta.id)
       const isProfileManager = contains(profileId, managerIds)
       const isProfileOwner = contains(profileId, ownerIds)
@@ -343,14 +344,13 @@ export default class Organization extends Component {
         }
       }
 
-      if (isProfileManager || isProfileOwner) {
-        profileActions.push({
-          name: 'Assign a Badge',
-          onClick: () => Router.push(
+      profileActions.push({
+        name: 'Assign a Badge',
+        onClick: () =>
+          Router.push(
             join(URL, `/organizations/${org.id}/badges/assign/${profileId}`)
           )
-        })
-      }
+      })
     }
 
     return (