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

chore(file-download): enable downloading folder/file contents from file directory #346

Conversation

priyakanabar-crest
Copy link
Contributor

This PR adds the ability to download the contents of a folder/file directly from the file directory.

zt_backend/router.py Outdated Show resolved Hide resolved
@Carson-Shaar
Copy link
Contributor

Will want to test this in dev as well

@priyakanabar-crest priyakanabar-crest force-pushed the enable-file-download-from-file-system branch 3 times, most recently from 3bfafa1 to 362e85b Compare October 22, 2024 06:46
@priyakanabar-crest
Copy link
Contributor Author

@Carson-Shaar I have updated the code and rebase the branch

@@ -79,6 +87,12 @@
</template>
</v-snackbar>

<FileFolderDownloadDialog
Copy link
Contributor

@Carson-Shaar Carson-Shaar Nov 12, 2024

Choose a reason for hiding this comment

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

All of the dialogs should live under the list item like this:

        <v-list-item
          v-if="<some_bool>"
          link
          prepend-icon="mdi-account-outline"
          title="Manage Access"
        >
          <ShareProjectDialog :project-name="item.name" />
        </v-list-item>

Note the link directive. Please refactor these as needed and use props in the dialogs

@Carson-Shaar
Copy link
Contributor

Please rebase this to resolve conflicts and test with very large folders

@priyakanabar-crest priyakanabar-crest force-pushed the enable-file-download-from-file-system branch 2 times, most recently from 362e85b to 571eb58 Compare November 15, 2024 11:51
@priyakanabar-crest
Copy link
Contributor Author

Rebased solve conflicts and updated downlader dialog to be in v-list-item

@Carson-Shaar Carson-Shaar force-pushed the enable-file-download-from-file-system branch from 394bcb1 to 87cbbdc Compare November 21, 2024 16:53
… add helper functions code and use it from there and removed unwnated code from the components
…tem ,updated dilaog for download and updated api path formation
@Carson-Shaar Carson-Shaar force-pushed the enable-file-download-from-file-system branch from 87cbbdc to 7107d61 Compare November 22, 2024 01:03
@Carson-Shaar Carson-Shaar merged commit 91dd35f into Zero-True:main Nov 22, 2024
14 of 18 checks passed
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.

2 participants