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

Version downloading/deleting #84

Merged
merged 8 commits into from
May 19, 2023
Merged

Version downloading/deleting #84

merged 8 commits into from
May 19, 2023

Conversation

kamorel
Copy link
Contributor

@kamorel kamorel commented May 16, 2023

Description

Allows specific object versions to be downloaded and deleted.

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@kamorel kamorel requested review from jujaga and TimCsaky as code owners May 16, 2023 20:03
@github-actions
Copy link

Coverage Report (Application)

Totals Coverage
Statements: 75% ( 51 / 68 )
Methods: 62.5% ( 5 / 8 )
Lines: 82.61% ( 38 / 46 )
Branches: 57.14% ( 8 / 14 )

@github-actions
Copy link

github-actions bot commented May 16, 2023

Coverage Report (Frontend)

Totals Coverage
Statements: 41.06% ( 404 / 984 )
Methods: 40% ( 92 / 230 )
Lines: 49.32% ( 288 / 584 )
Branches: 14.12% ( 24 / 170 )

@kamorel kamorel force-pushed the feature/version-actions branch from 8c37a97 to c1ae264 Compare May 16, 2023 20:31
@jujaga jujaga mentioned this pull request May 16, 2023
4 tasks
@kamorel kamorel force-pushed the feature/version-actions branch from e6bc0fe to 912c120 Compare May 16, 2023 22:47
@kamorel kamorel force-pushed the feature/version-actions branch from 912c120 to e08f32e Compare May 16, 2023 23:16
@kamorel kamorel force-pushed the feature/version-actions branch from e08f32e to b616c2e Compare May 18, 2023 21:57
frontend/src/store/objectStore.ts Outdated Show resolved Hide resolved
frontend/src/components/object/ObjectFileDetails.vue Outdated Show resolved Hide resolved
frontend/src/components/object/ObjectFileDetails.vue Outdated Show resolved Hide resolved
frontend/src/components/object/ObjectUploadBasic.vue Outdated Show resolved Hide resolved
frontend/src/components/object/ObjectUploadBasic.vue Outdated Show resolved Hide resolved
frontend/src/components/object/ObjectUploadBasic.vue Outdated Show resolved Hide resolved
frontend/src/components/object/DeleteObjectButton.vue Outdated Show resolved Hide resolved
toast.success('File deleted');
await versionStore.fetchVersions({ objectId: props.objectId });
Copy link
Contributor

Choose a reason for hiding this comment

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

i think i found when deleting any version except latest, that version remains in the table until you do a hard refresh.
maybe the fetchVersions() needs to do that partition logic somewhere to remove that old version from the store (?)

Copy link
Contributor

Choose a reason for hiding this comment

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

or maybe it's because the route isnt changing 🤔

style="display: none"
accept="*"
@change="onChange"
/>
Copy link
Contributor

@TimCsaky TimCsaky May 18, 2023

Choose a reason for hiding this comment

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

FYI. Sharolyn mentioned that a 'uploading progress indicator' would be good.
i would have thought it would be ok to tackle that later, along with showing a something similar in the main upload panel. realistically people aren't going to be updating big files anytime soon.
... if we did want to do something short term.. there might be a a quick win if we could put in something here like:
<Spinner v-if"!done" /> (using a flag in the onUpload() function)

@@ -142,24 +146,28 @@ watch( [props], () => {
v-if="data.public || permissionStore.isObjectActionAllowed(
Copy link
Contributor

Choose a reason for hiding this comment

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

just a thought..i think they can't even get to the file details page if they havent got READ permission. but if we re-use this component, then this is good.

</Column>
</DataTable>
</div>
<GridRow
Copy link
Contributor

Choose a reason for hiding this comment

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

i think this is good.. not sure datatable was neccessary

@kamorel kamorel force-pushed the feature/version-actions branch from 3af2d07 to 397a0ff Compare May 19, 2023 19:27
@TimCsaky TimCsaky merged commit 9d58385 into master May 19, 2023
@jujaga jujaga deleted the feature/version-actions branch May 19, 2023 20:20
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