-
Notifications
You must be signed in to change notification settings - Fork 141
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
Fix/loglink moved below logs table #1939
Fix/loglink moved below logs table #1939
Conversation
Maybe keep the link also in the about tab for backwards compatibility? |
This reverts commit 855f295.
…om/shivansh02/vorta into fix/loglink-moved-below-logs-table
Added the link back to the about tab, please take a look. |
Didn't run it locally, but looks fine. Also doesn't change up all the UI files. Will merge tomorrow after testing locally. |
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.
Fixes #1862 |
Thanks for contributing! Active community members like yourself ensure the quality and relevance of this project. |
Moved the complete logs link from about tab to below logs table.
Description
Removed the link from about tab and added it below logs table on the schedule tab.
Related Issue
#1862
Motivation and Context
Schedule->Logs is a better location for the link, so that one can view the complete logs in case the ones in the summary aren't enough.
How Has This Been Tested?
Tested by me locally.
Screenshots (if appropriate):
Types of changes
Checklist:
I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.