Skip to content

Conversation

@iaigner
Copy link
Contributor

@iaigner iaigner commented Oct 6, 2025

  • fixed problem with primevue loading animations
  • added loading spinner to download study data button and downlaod auditlog button
  • added new confirmation dialog for navigation change
  • called new confirmation dialog when user wants do navigate from download study data page to another if downlaod buffer is still going on

note: download functionality will continue, even when user navigates away but some edit functions are locked until the download is completed. the progressbar indicator will show until download starts.

Bildschirmaufnahme.2025-10-06.um.15.29.56.mov

Ticket: #617
Assignee: Isabella Aigner

@iaigner iaigner requested a review from markus-ebner October 6, 2025 13:46
@iaigner iaigner changed the title Prehab2rehab/617 loading indicator for downloading study data 617 loading indicator for downloading study data Oct 6, 2025
@janoliver20 janoliver20 self-requested a review October 7, 2025 05:30
@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit e1356b8. ± Comparison against base commit 47dd299.

@iaigner iaigner requested a review from janoliver20 October 7, 2025 08:54
Copy link
Contributor

@janoliver20 janoliver20 left a comment

Choose a reason for hiding this comment

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

LGTM

@iaigner iaigner changed the title 617 loading indicator for downloading study data 617: loading indicator for downloading study data Oct 7, 2025
@iaigner iaigner merged commit a7f1132 into develop Oct 7, 2025
7 checks passed
@iaigner iaigner deleted the prehab2rehab/617-loading-indicator-for-downloading-study-data branch October 7, 2025 08:59
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