Add Windows app download button to quickstart page#22
Conversation
Serve the bundled OpenMS-App.zip (downloaded into /app by the Dockerfile) as a cached, server-wide resource via st.download_button on the DDA-LFQ quickstart page. The button is hidden when the asset is not present (e.g. local dev runs without the bundled build).
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThe quickstart page now includes conditional rendering of a download button for Changes
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.Comment |
Summary
Added functionality to the quickstart page to allow users to download a Windows version of the OpenMS application directly from the Streamlit interface.
Key Changes
Pathfrompathlibfor file system operationsload_windows_app_bytes()that reads the Windows app binary from/app/OpenMS-App.zipif it existsImplementation Details
@st.cache_resourcedecorator to cache the app bytes in memory, avoiding repeated file readsNoneif the app file doesn't exist, preventing errors in environments where the Windows app isn't availablehttps://claude.ai/code/session_01QmHobjk1tgB7i5iGY5fAoE
Summary by CodeRabbit