Merge pull request #680 from eco-stake/handle-proposals-without-content

Handle proposals with no content
eco-stake · Nov 25, 2022 · 61a48ed · 61a48ed
2 parents 041a959 + fdfff66
commit 61a48ed
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/components/Proposals.js b/src/components/Proposals.js
@@ -81,7 +81,7 @@ function Proposals(props) {
         <td className="d-none d-md-table-cell">{proposalId}</td>
         <td>
           <span role="button" onClick={() => props.showProposal(proposal)}>
-            {proposal.content.title}
+            {proposal.title}
           </span>
         </td>
         <td className="d-none d-sm-table-cell text-center text-nowrap">

diff --git a/src/utils/Proposal.mjs b/src/utils/Proposal.mjs
@@ -9,18 +9,19 @@ export const PROPOSAL_STATUSES = {
 
 const Proposal = (data) => {
 
-  const { title, description } = data.content
-  const type = data.content['@type']
+  const { title, description } = data.content || {}
+  const type = data.content && data.content['@type']
 
-  const fixedDescription = description.split(/\\n/).join('\n')
+  const fixedDescription = description && description.split(/\\n/).join('\n')
   const statusHuman = PROPOSAL_STATUSES[data.status]
-  const typeHuman = type.split('.').reverse()[0]
+  const typeHuman = type ? type.split('.').reverse()[0] : 'Unknown'
 
   const isDeposit = data.status === 'PROPOSAL_STATUS_DEPOSIT_PERIOD'
   const isVoting = data.status === 'PROPOSAL_STATUS_VOTING_PERIOD'
 
   return {
     ...data,
+    title,
     type,
     fixedDescription,
     statusHuman,