Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Expo open native file system and AWS bucket storage #17

Merged
merged 31 commits into from
Jan 31, 2024

Conversation

oliviaseds
Copy link
Contributor

Description

Expo Open Native File System
File Storage

Set up native file system selector to upload documents using Expo DocumentPicker library.
Connected file upload and deletion to AWS bucket to store file content.

How Has This Been Tested?

Tested APIs with Postman and by manually uploading different file types.

Checklist

  • I have performed a self-review of my code
  • I have reached out to another developer to review my code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes

// @tags file
// @success 204
// @router /files/{fname} [delete]
func (pg *PgModel) DeleteFileRoute(c *gin.Context) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this working? Could prob be removed if not we just care about uploading :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh really - I didn't realize that. Deleting hasn't been tested but could be, we would just need to make sure that files are removed from the bucket on your end.

@@ -12,14 +12,17 @@
},
"dependencies": {
"@react-navigation/native": "^6.1.9",
"@tanstack/react-query": "^5.17.19",
Copy link
Contributor

Choose a reason for hiding this comment

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

This package isnt used could prob be removed as well

Copy link
Contributor

@MattCMcCoy MattCMcCoy left a comment

Choose a reason for hiding this comment

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

@wyattchris @oliviaseds Amazing Job Guys Yall are Goats

Can you make sure you hit the Squash button while merging, Turn those 31 commits into one with all the details of what yall did :)
It may be hidden in the arrow
Screenshot 2024-01-30 at 3 28 09 PM

@oliviaseds oliviaseds merged commit 99cecb3 into main Jan 31, 2024
3 checks passed
@MattCMcCoy MattCMcCoy deleted the feature-file-storage branch January 31, 2024 03:26
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.

3 participants