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

Export of inventory lists should be impossible if there are too many items. #1329

Closed
pronguen opened this issue Oct 26, 2020 · 2 comments · Fixed by #1842
Closed

Export of inventory lists should be impossible if there are too many items. #1329

pronguen opened this issue Oct 26, 2020 · 2 comments · Fixed by #1842
Labels
enhancement Improvement of an existing feature f: professional ui Professional interface p-Low Low priority stale Stale or no longer relevant

Comments

@pronguen
Copy link
Contributor

How it works

The export of inventory lists gives a time out if there are too many items.

Improvement suggestion

The export button should be disabled (with a message) if too many items are in the list, in order to avoid the time out and inform the user.
@pronguen pronguen added enhancement Improvement of an existing feature triage labels Oct 26, 2020
@iGormilhit iGormilhit added f: professional ui Professional interface p-Low Low priority labels Nov 26, 2020
@iGormilhit
Copy link

Related to #1456

@iGormilhit iGormilhit mentioned this issue Nov 26, 2020
Closed
@iGormilhit iGormilhit removed the triage label Nov 26, 2020
lauren-d added a commit to lauren-d/rero-ils that referenced this issue May 5, 2021
@lauren-d
items: improve response time for csv export
079ac0b 
The CSV export of the inventory list has an important waiting time.
With this commit, we try to improve the response time.

* Uses ciso8601 to parse date fields instead `format_date_filter`.
* Adds new api endpoint for inventory list.
* Uses streaming to process download csv file.
* Uses bunch request to reduce elasticsearch calls.
* Adds some fields in csv file.
* Closes rero#1456.
* Closes rero#1329.

Co-Authored-by: Laurent Dubois <laurent.dubois@itld-solutions.be>
@lauren-d lauren-d mentioned this issue May 5, 2021
Merged
7 tasks
lauren-d added a commit to lauren-d/rero-ils that referenced this issue May 11, 2021
@lauren-d
items: improve response time for csv export
01beedd 
The CSV export of the inventory list has an important waiting time.
With this commit, we try to improve the response time.

* Uses ciso8601 to parse date fields instead `format_date_filter`.
* Adds new api endpoint for inventory list.
* Uses streaming to process download csv file.
* Uses bunch request to reduce elasticsearch calls.
* Adds some fields in csv file.
* Closes rero#1456.
* Closes rero#1329.

Co-Authored-by: Laurent Dubois <laurent.dubois@itld-solutions.be>
@github-actions
Copy link

This issue is stale because it has been open 6 months with no activity.

@github-actions github-actions bot added the stale Stale or no longer relevant label May 26, 2021
lauren-d added a commit to lauren-d/rero-ils that referenced this issue May 31, 2021
@lauren-d
items: improve response time for csv export
bb0fffd 
The CSV export of the inventory list has an important waiting time.
With this commit, we try to improve the response time.

* Uses ciso8601 to parse date fields instead `format_date_filter`.
* Adds new api endpoint for inventory list.
* Uses streaming to process download csv file.
* Uses bunch request to reduce elasticsearch calls.
* Adds some fields in csv file.
* Closes rero#1456.
* Closes rero#1329.

Co-Authored-by: Laurent Dubois <laurent.dubois@itld-solutions.be>
lauren-d added a commit that referenced this issue May 31, 2021
@lauren-d
items: improve response time for csv export
45d8fd8 
The CSV export of the inventory list has an important waiting time.
With this commit, we try to improve the response time.

* Uses ciso8601 to parse date fields instead `format_date_filter`.
* Adds new api endpoint for inventory list.
* Uses streaming to process download csv file.
* Uses bunch request to reduce elasticsearch calls.
* Adds some fields in csv file.
* Closes #1456.
* Closes #1329.

Co-Authored-by: Laurent Dubois <laurent.dubois@itld-solutions.be>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Improvement of an existing feature f: professional ui Professional interface p-Low Low priority stale Stale or no longer relevant
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

items: improve response time for csv export lauren-d/rero-ils
2 participants
@iGormilhit @pronguen