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

Merge Zip files development into main #1848

Merged
merged 26 commits into from
Dec 4, 2024
Merged

Merge Zip files development into main #1848

merged 26 commits into from
Dec 4, 2024

Conversation

bbpennel
Copy link
Member

@bbpennel bbpennel commented Dec 3, 2024

No description provided.

sharonluong and others added 26 commits August 20, 2024 08:59
* BXC-4673 start bulk download controller

* BXC-4673 returning a file in the controller and a test

* BXC-4673 DRY up to share logic between download controller tests

* BXC-4673 finish bulk download controller test

* BXC-4673 cleanup

* BXC-4673 update directory name

* BXC-4673 whoops the __files naming was necessary:

* BXC-4673 adding object type mismatch handling
* BXC-4684 add download zip button

* BXC-4684 add tests
* Move bulk download above the file list

* Bump bulma version

* Use bulma without a dark mode

* Align search box with zip download button

* Use bulma's button layout

* Fix issues with hompage hover links

* Change button text, per RCMT
* Move bulkDownload to its own component

* Add service to retrieve the total filesize of all files in a work

* Fix contact link

* Use fileSizeSort to calculate total file size

* Fix security issues

* Use bulma button layout

* Switch to formatting file size on the client side
* Add a jump to download button

* Fix spacing issues with jump button

* Fix spacing of reset button and search box for file list
* BXC-4771 add download file limit

* BXC-4771 cleanup

* BXC-4771 set file limit in test

* BXC-4771 DRY up code and update limit

* BXC-4771 set limit to 99 for 100 files downloaded

* BXC-4771 set limit to 100 for 100 files downloaded
* Disentangling bulk download from file list to make it easier to get access to needed data

* Styling updates to button

* Limit zip download to authenticated or oncampus users. Server returns a property if the user has permission

* Fix test

* Fixing up style issues and getting more parts working with bulma 1.0

* Fix test and update readme

* Convert search clearing links to buttons
* Add alert which prompts user to confirm they want to go through with export when there are too many files

* Update bulkDownload.vue

* Update bulkDownload.spec.js
…ays the case for containers. Fix nesting of briefObject within response objects in tests, since sometimes it was nested, double nested, or not nested. restrictedFiles now uses restrictedContent to DRY it up, and restrictedContent can now also deal with data being nested or not
* Bxc 4734 thumbnail placeholder (#1830)

* BXC-4734 update for thumbnail placeholder usage

* BXC-4734 remove placeholder test from download image IT

* BXC-4734 change null to empty array

* BXC-4734 clean up code

* BXC-4734 update file extension

* Block jp2 processing for x-raw-panasonic. Add unit test (#1829)

* Bxc 4761 replace existing bug (#1832)

* BXC-4761 set force to true to force access copy regeneration

* BXC-4761 set to string true

* Bxc 4760 admin unit thumb (#1833)

* BXC-4760 excuse admin unit from permissions check

* BXC-4760 move method

* Display filesize on file objects

* Move file downloads into the files table (#1831)

* Move image download access into the files table

* Update tests and fix issues found by tests

* Fix issue with empty download column showing up, if there is no downloadable content

* Add translations back in and check dropdown urls, instead of text, as translations aren't in scope in the tests.

* Don't show modal if user is logged in

* Check for restricted content, not downloadable images, and pass the file's info into the restriction check instead of using the work's info.

* Fix display issue and modal tests

* Use login modal on file pages

* Don't show modal login if logging in doesn't grant further access.

* Fix layout of buttons on small screens and add "view" item back in for file records

* Display login button for folders and collections

* Fix button layout

* Fix button layout in the files table

* Fix issue with no valid download sizes, but user canViewOriginal

* Set default as logged in

* Check if JP2 exists and rename restrictedContent component

* block jp2 processing for image/vnd.microsoft.icon and add test (#1837)

* Fix merge of changes in restricted content into object actions

* Fixed bug when format field not present on brief record, which is always the case for containers. Fix nesting of briefObject within response objects in tests, since sometimes it was nested, double nested, or not nested. restrictedFiles now uses restrictedContent to DRY it up, and restrictedContent can now also deal with data being nested or not

* Only allow view button on files

* Fixes to spacing of restricted content sections

---------

Co-authored-by: sharonl <[email protected]>
Co-authored-by: lfarrell <[email protected]>
Co-authored-by: krwong <[email protected]>
…elopment-merge

Revert "Merge main into Zip files development"
…utton now that it contains spans. Fix file list cutting off download dropdown for the last entry, and horizontal scroll
@bbpennel bbpennel merged commit c177bb7 into main Dec 4, 2024
6 checks passed
@bbpennel bbpennel deleted the zip-files-development branch December 4, 2024 15: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