Skip to content

Adding a how to choose storage section on data management index page#949

Merged
wwarriner merged 4 commits into
uabrc:mainfrom
bdu-birhanu:feat-dm-index
Apr 18, 2025
Merged

Adding a how to choose storage section on data management index page#949
wwarriner merged 4 commits into
uabrc:mainfrom
bdu-birhanu:feat-dm-index

Conversation

@bdu-birhanu
Copy link
Copy Markdown
Contributor

Pull Request

Overview

Proposed Changes

  • Added a how to choose the right Storage section on the data management index section.
  • Updated the backups section on the data management index page

Related Issues

Fixes #904

@bdu-birhanu bdu-birhanu added the pr: review PR is ready for review label Mar 27, 2025
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

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

Great start! I have a question for you, and some thoughts on how to provide some "escape hatches" and generally guide people to improve their practices.

Comment thread docs/data_management/index.md
Comment thread docs/data_management/index.md Outdated
Comment thread docs/data_management/index.md
Comment thread docs/data_management/index.md
Comment thread docs/data_management/index.md Outdated
Comment thread docs/data_management/index.md Outdated
Comment thread docs/data_management/index.md
@wwarriner wwarriner added pr: changes requested Review complete, needs changes and removed pr: review PR is ready for review labels Mar 28, 2025
@bdu-birhanu bdu-birhanu added pr: review PR is ready for review and removed pr: changes requested Review complete, needs changes labels Apr 2, 2025
Copy link
Copy Markdown
Contributor

@wwarriner wwarriner left a comment

Choose a reason for hiding this comment

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

Looks great, one tiny change to grammar. When fixed please change label to "merge".

Comment thread docs/data_management/index.md Outdated
@bdu-birhanu bdu-birhanu added pr: merge PR is ready to merge and removed pr: review PR is ready for review labels Apr 18, 2025
@wwarriner wwarriner merged commit 79982f3 into uabrc:main Apr 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: merge PR is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data Management index improvement

2 participants