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.

Sign up for GitHub

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

Jump to bottom

A more flexible file model + data file upload #510

Closed
wants to merge 68 commits into from
Closed

A more flexible file model + data file upload #510

wants to merge 68 commits into from

Conversation

sou-chon
Copy link
Collaborator

@sou-chon sou-chon commented Jun 3, 2020

  • change to a more flexible file model to accommodate different kinds of files
  • allowing directory structure
  • upload data file that blob data to a patient
  • user can have their own files too (so in the future they can create scripts)

@sou-chon sou-chon marked this pull request as draft June 3, 2020 09:10
@sou-chon sou-chon linked an issue Jun 3, 2020 that may be closed by this pull request
blob patient data #499
Open
@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2020

This pull request introduces 14 alerts and fixes 1 when merging c0f3792 into ea1bf61 - view on LGTM.com

new alerts:

  • 5 for Superfluous trailing arguments
  • 4 for Syntax error
  • 4 for Unused variable, import, function or class
  • 1 for Useless assignment to property

fixed alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2020

This pull request introduces 14 alerts and fixes 1 when merging 99895e5 into d08b388 - view on LGTM.com

new alerts:

  • 5 for Superfluous trailing arguments
  • 4 for Syntax error
  • 4 for Unused variable, import, function or class
  • 1 for Useless assignment to property

fixed alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2020

This pull request introduces 10 alerts when merging 1169615 into 67e1835 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Syntax error
  • 1 for Useless conditional
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 10 alerts when merging a2a3d94 into 7fae692 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Syntax error
  • 1 for Useless conditional
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 12 alerts when merging e48f7f2 into d58e334 - view on LGTM.com

new alerts:

  • 8 for Unused variable, import, function or class
  • 2 for Syntax error
  • 1 for Useless conditional
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jun 18, 2020

This pull request introduces 2 alerts when merging 354ab0b into 44d9e92 - view on LGTM.com

new alerts:

  • 1 for Useless conditional
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jun 28, 2020

This pull request introduces 2 alerts when merging 1087134 into b402d8b - view on LGTM.com

new alerts:

  • 1 for Useless conditional
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jun 30, 2020

This pull request introduces 2 alerts when merging bd02b69 into 0160a3a - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jul 1, 2020

This pull request introduces 2 alerts when merging 9029089 into 0ef16ef - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jul 3, 2020

This pull request introduces 1 alert when merging 7a79179 into 887b78e - view on LGTM.com

new alerts:

  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jul 3, 2020

This pull request introduces 1 alert when merging db6b692 into 887b78e - view on LGTM.com

new alerts:

  • 1 for Useless assignment to property

@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2020

This pull request introduces 12 alerts when merging 35ba26f into e0a0d0e - view on LGTM.com

new alerts:

  • 12 for Syntax error

@sou-chon sou-chon closed this Mar 12, 2021
@sou-chon sou-chon deleted the feat/upload-blobs branch March 12, 2021 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

blob patient data
1 participant
@sou-chon