delete commented code and move into branch 152-request-attachments

Author: summer-cook

src/components/FilesTable/FilesTable.jsx

@@ -6,9 +6,6 @@ import {
 } from 'react-bootstrap'
 import { allowNull } from '../../resources/utilityFunctions'
 
-// TODO(summer-cook):
-// add back handleDeleteFile to props once posting attachments/messages is working/during the work for this ticket:
-// https://github.com/scientist-softserv/webstore-component-library/issues/152*/
 const FilesTable = ({ addClass, files }) => {
   if (files.length === 0) {
     return (
@@ -24,9 +21,6 @@ const FilesTable = ({ addClass, files }) => {
           <th>Uploaded By</th>
           <th>Size</th>
           <th>Created At</th>
-          {/* TODO(summer-cook):
-            - comment this back in, see comment above */}
-          {/* <th> </th> */}
         </tr>
       </thead>
       <tbody>
@@ -38,16 +32,6 @@ const FilesTable = ({ addClass, files }) => {
               <td>{uploadedBy}</td>
               <td>{contentLength}</td>
               <td>{createdAt}</td>
-              {/* TODO(summer-cook):
-                  - comment this back in (see above comment)
-                  - add an X icon here instead of remove text, make sure the handleDeleteFile is working */}
-              {/* <td>
-                    <Button
-                      primary
-                      onClick={handleDeleteFile}>
-                        Remove
-                    </Button>
-                  </td> */}
             </tr>
           )
         })}
@@ -71,8 +55,6 @@ FilesTable.propTypes = {
       uuid: PropTypes.string.isRequired,
     }),
   ).isRequired,
-  // TODO(summer-cook): add this back in. see comment above.
-  // handleDeleteFile: PropTypes.func.isRequired,
 }
 
 FilesTable.defaultProps = {

src/compounds/ActionsGroup/ActionsGroup.jsx

@@ -8,7 +8,7 @@ import { allowNull } from '../../resources/utilityFunctions'
 import './actions-group.scss'
 
 // TODO: add back handleSendingMessagesOrFiles into the props. see below TODO.
-const ActionsGroup = ({ initialFiles }) => {
+const ActionsGroup = ({ initialFiles, handleSendingMessagesOrFiles }) => {
   const [show, setShow] = useState(false)
   const [action, setAction] = useState(null)
 
@@ -60,7 +60,7 @@ const ActionsGroup = ({ initialFiles }) => {
 }
 
 ActionsGroup.propTypes = {
-  // handleSendingMessagesOrFiles: PropTypes.func.isRequired,
+  handleSendingMessagesOrFiles: PropTypes.func.isRequired,
   initialFiles: PropTypes.arrayOf(
     PropTypes.shape({
       contentLength: PropTypes.string.isRequired,

src/compounds/ActionsGroup/actions/ViewFiles.jsx

@@ -1,28 +1,15 @@
 import React
-  // { useState, useRef } TODO comment back in
   from 'react'
 import PropTypes from 'prop-types'
 import {
-  // Button,
-  // CloseButton,
-  // Form,
-  // InputGroup,
-  // ListGroup,
   Offcanvas,
   Tab,
   Tabs,
 } from 'react-bootstrap'
-// import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'
 import FilesTable from '../../../components/FilesTable/FilesTable'
-// import { apiV2CompatibleStrings, convertToBase64 } from '../../../resources/utilityFunctions'
 import { allowNull } from '../../../resources/utilityFunctions'
 
 const ViewFiles = ({ initialFiles, handleClose }) => {
-  // TODO(summercook):
-  // - comment back in the following 2 lines & above imports once posting messages/attachments is working
-  // const fileRef = useRef(null)
-  // const [files, setFiles] = useState([])
-
   const documentTabs = [
     {
       eventKey: 'files',
@@ -36,71 +23,12 @@ const ViewFiles = ({ initialFiles, handleClose }) => {
     },
   ]
 
-  // TODO(summercook):
-  // - comment back in the following 3 methods once posting messages/attachments is working
-  // may need to use the handleSendingMessagesOrFiles to post
-  // const handleSubmit = (event) => {
-  //   event.preventDefault()
-  //   onSubmit({ files: apiV2CompatibleStrings(files) })
-  // }
-
-  // const handleAddFile = async (event) => {
-  //   event.preventDefault()
-  //   try {
-  //     // "event.target.files" returns a FileList, which looks like an array but does not respond to array methods
-  //     // except "length". we are using the spread syntax to set "files" to be an iterable array
-  //     const fileArray = [...event.target.files]
-  //     const newBase64Files = await Promise.all(convertToBase64(fileArray))
-  //     const newFiles = fileArray.map((file, index) => ({ [file.name]: newBase64Files[index] }))
-
-  //     setFiles([...files, ...newFiles])
-  //     fileRef.current.value = ''
-  //   } catch (error) {
-  //     throw new Error(error)
-  //   }
-  // }
-
-  // const handleDeleteFile = (file) => {
-  //   const remainingFiles = files.filter((obj) => obj !== file)
-  //   setFiles(remainingFiles)
-  // }
   return (
     <Offcanvas show onHide={handleClose} placement='end' scroll='true'>
       <Offcanvas.Header className='d-flex border-bottom px-3 py-2 bg-light' closeButton>
         <Offcanvas.Title>Documents</Offcanvas.Title>
       </Offcanvas.Header>
       <Offcanvas.Body className='border rounded p-2 m-3'>
-        {/* TODO(summercook): see above comment */}
-        {/* <Form>
-            <h6 className='mt-3'>Upload Additional Documents</h6>
-            <InputGroup controlId='attachments' className='mb-3'>
-            <Form.Control
-            multiple
-            type='file'
-            onChange={handleAddFile}
-            ref={fileRef}
-            />
-            <Button
-            variant='outline-primary'
-            onClick={handleSubmit}
-            type='submit'
-            >
-            <FontAwesomeIcon icon='fa-upload' />
-            </Button>
-            </InputGroup>
-            </Form>
-            <ListGroup variant='flush'>
-            {files.map((file) => {
-              const fileName = Object.keys(file)[0]
-
-              return (
-                <ListGroup.Item key={fileName} className='d-flex align-items-center'>
-                <span>{fileName}</span>
-                <CloseButton onClick={() => handleDeleteFile(file)} className='ms-auto' />
-                </ListGroup.Item>
-                )
-              })}
-            </ListGroup> */}
         <Tabs defaultActiveKey='files' id='document-tabs' justify fill>
           {documentTabs && documentTabs.map((tab) => {
             const { eventKey, title, status } = tab
@@ -115,8 +43,6 @@ const ViewFiles = ({ initialFiles, handleClose }) => {
                 <FilesTable
                   files={filteredFiles}
                   status={status}
-                  // TODO(summercook): see above comment
-                  // handleDeleteFile={handleDeleteFile}
                 />
               </Tab>
             )