-
Notifications
You must be signed in to change notification settings - Fork 33
[PB-5465] Implement file version history actions #1771
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
Merged
terrerox
merged 32 commits into
feature/file-version-history
from
feature/file-version-history-v2
Jan 10, 2026
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
1612aff
feature: implement file version history with restore, delete, and d…
terrerox 4a2ab35
feature: enhance internationalization for version management with add…
terrerox f59d219
refactor: update skeleton component to use Array.from for better read…
terrerox e348210
feature: improve file version downloads with multipart support and …
terrerox 7195005
fix: update @internxt/sdk version to 1.11.19 in package.json and yarn…
terrerox e2b34bb
refactor: remove redundant error handling in version download
terrerox da946c8
Merge pull request #1773 from internxt/feature/file-version-history-v3
terrerox f3a74ba
feature: enhance version history with batch operations and SDK inte…
terrerox fb6a5fd
feature: refactor file upload logic and enhance version info handling…
terrerox 0278fd6
feature: update currentVersion state handling in Sidebar to reflect i…
terrerox 7f94127
feature: integrate versioning limits context and enhance version hi…
terrerox 9d6956e
feature: implement versioning limits context and enhance file replace…
terrerox 2c5b00a
refactor: implement Redux-based caching for file versions and remov…
terrerox 79b26db
feature: enhance version history menu item to handle availability bas…
terrerox e3bffd7
refactor: improve file versions state management and naming convent…
terrerox 02fe0fe
refactor: remove unused selectors from fileVersionsSelectors
terrerox 700735a
test: add comprehensive test coverage for file version history feature
terrerox abe4124
test: add unit tests for Redux file versions slice and version item a…
terrerox f3f12d1
refactor: simplify test descriptions in file versions slice tests
terrerox d919cd2
refactor: add default empty array return to getVersionsByFileId selector
terrerox ea961f7
chore: update @internxt/sdk package registry source
terrerox 564699e
test: improve test descriptions using Given-When-Then pattern
terrerox 7084646
test: refactor test descriptions to use Given-When-Then pattern
terrerox 0a418b4
test: refine test descriptions for consistency and clarity
terrerox 6906b89
feature: add file extension validation for version replacement and im…
terrerox 2bf1abf
refactor: add user avatar support and update border styling in versio…
terrerox e361615
fix: add backdrop overlay to prevent dropdown hover bleed-through in …
terrerox 467836c
fix: pin version size label to the right edge
terrerox 73de535
Merge pull request #1785 from internxt/feature/file-version-history-v6
terrerox 59796c9
Merge pull request #1784 from internxt/feature/file-version-history-v5
terrerox 8182d3b
Merge pull request #1779 from internxt/feature/file-version-history-v4
terrerox 34d61ea
Merge pull request #1774 from internxt/feature/file-version-history-v3
terrerox File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -892,7 +892,20 @@ | |
| "autosaveVersions": "{{count}}/{{total}} autosave versions", | ||
| "restoreVersion": "Restore version", | ||
| "downloadVersion": "Download version", | ||
| "deleteVersion": "Delete version" | ||
| "deleteVersion": "Delete version", | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The remaining translations are included in this PR: #1773 |
||
| "deleteVersionTitle": "Delete version", | ||
| "deleteButton": "Delete", | ||
| "restoreButton": "Restore", | ||
| "downloadError": "Failed to download version", | ||
| "deleteVersionAdvice": "This version will be permanently deleted. \nThis action cannot be undone.", | ||
| "deletingVersion": "Deleting", | ||
| "restoreVersionTitle": "Restore version", | ||
| "restoreVersionAdvice": "Restoring this version will replace the current file and remove all newer versions. \nOnce restored, this action cannot be undone.\n\nYou can download a copy of newer versions before restoring.", | ||
| "restoringVersion": "Restoring", | ||
| "restoreSuccess": "Version restored successfully", | ||
| "restoreError": "Failed to restore version", | ||
| "deleteSuccess": "Version deleted successfully", | ||
| "deleteError": "Failed to delete version" | ||
| }, | ||
| "shareModal": { | ||
| "title": "Share \"{{name}}\"", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
16 changes: 16 additions & 0 deletions
16
src/app/store/slices/fileVersions/fileVersions.selectors.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| import { FileVersion, FileLimitsResponse } from '@internxt/sdk/dist/drive/storage/types'; | ||
| import { RootState } from '../..'; | ||
|
|
||
| const fileVersionsSelectors = { | ||
| getLimits(state: RootState): FileLimitsResponse | null { | ||
| return state.fileVersions.limits; | ||
| }, | ||
| getVersionsByFileId(state: RootState, fileId: NonNullable<FileVersion['fileId']>): FileVersion[] { | ||
| return state.fileVersions.versionsByFileId[fileId] ?? []; | ||
| }, | ||
| isLoadingByFileId(state: RootState, fileId: NonNullable<FileVersion['fileId']>): boolean { | ||
| return state.fileVersions.isLoadingByFileId[fileId] ?? false; | ||
| }, | ||
| }; | ||
|
|
||
| export default fileVersionsSelectors; |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this div wrapping the task logger in necessary? I think could cause wrong behaviours
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When downloading a version, the task logger was located under the File Versions sidebar, making it difficult to cancel the download.