Skip to content

Add ability to attach files in CCE Minor Proposals#1418

Merged
bledsoef merged 16 commits intodevelopmentfrom
1411_OtherEngagementFiles
Apr 8, 2025
Merged

Add ability to attach files in CCE Minor Proposals#1418
bledsoef merged 16 commits intodevelopmentfrom
1411_OtherEngagementFiles

Conversation

@bledsoef
Copy link
Copy Markdown
Contributor

@bledsoef bledsoef commented Feb 24, 2025

Issue Description

Fixes #1411

  • Files we attached to the otherEngagement page were not able to be stored.

Changes

  • Utilized the attachmentUpload modal to store the file image.
  • Created a proposalid to distinguish between the different types of images that will be uploaded.

Testing

  • Checkout 1411_OtherEngagementFiles
  • Navigate to Minor Management in the Admin sidebar dropdown.
  • Click on the user ayisie to navigate to their CCE Minor Profile
  • Switch to the Manage Proposals tab at the top.
  • Click on the request Other Engagement button and fill out the information. Be sure to add a file.
  • Check the database in attachmentuploads to see if the file was stored.
  • It should look something like this,
    Screenshot 2025-02-27 164358
    . The 1 represents the id (image 1 ) of the new file we uploaded.

@bledsoef bledsoef changed the title 1411 other engagement files Able to attach files in CCE Minor Proposals Mar 10, 2025
@bledsoef bledsoef changed the title Able to attach files in CCE Minor Proposals Add ability to attach files in CCE Minor Proposals Mar 14, 2025
@bledsoef bledsoef changed the base branch from development to 1395_NewCCEMinorPortal March 14, 2025 18:13
Base automatically changed from 1395_NewCCEMinorPortal to development April 4, 2025 20:23
Copy link
Copy Markdown
Contributor

@MImran2002 MImran2002 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good. It is saved within the database.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2025

View Code Coverage

@bledsoef bledsoef merged commit 1e270f7 into development Apr 8, 2025
5 checks passed
@bledsoef bledsoef deleted the 1411_OtherEngagementFiles branch April 8, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get files working with Other Engagement

3 participants