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.

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

Rough draft of updated customize article. #1231

Merged
merged 27 commits into from
Apr 12, 2021
Merged

Rough draft of updated customize article. #1231

merged 27 commits into from
Apr 12, 2021

Conversation

lanceleonard
Copy link
Contributor

This is an in-progress update. Screen shots need to be added and additional content changes are expected.

@lanceleonard lanceleonard marked this pull request as draft February 4, 2021 23:48
@lanceleonard lanceleonard self-assigned this Feb 5, 2021
Copy link
Contributor

@kaitlynmichael kaitlynmichael left a comment

Choose a reason for hiding this comment

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

Overall, I think this is a VAST improvement.

content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
Copy link
Contributor

@banker banker left a comment

Choose a reason for hiding this comment

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

@lanceleonard This is looking good to me!

@lanceleonard lanceleonard marked this pull request as ready for review March 1, 2021 03:56
@lanceleonard lanceleonard requested a review from joniredis March 1, 2021 03:58
@lanceleonard
Copy link
Contributor Author

This PR starts reworking the Administration section. Two articles are affected:

Screenshots have been created/updated to reflect UX changes.

New PRs will cover the other tasks shortly.

Copy link
Contributor

@kaitlynmichael kaitlynmichael left a comment

Choose a reason for hiding this comment

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

I added a few suggestions, but it looks great!

content/rc/administration/setup/_index.md Outdated Show resolved Hide resolved
content/rc/administration/setup/create-subscription.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
content/rc/administration/setup/customize-flexible-plan.md Outdated Show resolved Hide resolved
@lanceleonard lanceleonard changed the base branch from master to release-rc-new-ui March 16, 2021 17:22
@lanceleonard lanceleonard changed the base branch from release-rc-new-ui to master April 7, 2021 15:17
Copy link
Contributor

@kaitlynmichael kaitlynmichael left a comment

Choose a reason for hiding this comment

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

Looks good to me!

banker
banker previously requested changes Apr 9, 2021
Copy link
Contributor

@banker banker left a comment

Choose a reason for hiding this comment

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

Very nicely done, @lanceleonard. A big improvement!

content/rc/administration/setup/_index.md Outdated Show resolved Hide resolved
content/rc/databases/monitor-performance.md Outdated Show resolved Hide resolved
content/rc/databases/flush-data.md Outdated Show resolved Hide resolved
content/rc/databases/monitor-performance.md Outdated Show resolved Hide resolved
content/rc/databases/monitor-performance.md Outdated Show resolved Hide resolved
content/rc/subscriptions/create-flexible-subscription.md Outdated Show resolved Hide resolved
content/rc/subscriptions/delete-subscription.md Outdated Show resolved Hide resolved
content/rc/subscriptions/view-fixed-subscription.md Outdated Show resolved Hide resolved
content/rc/subscriptions/view-fixed-subscription.md Outdated Show resolved Hide resolved
content/rc/subscriptions/view-flexible-subscription.md Outdated Show resolved Hide resolved
@lanceleonard lanceleonard dismissed banker’s stale review April 12, 2021 21:52

Changes incorporated

@lanceleonard lanceleonard merged commit 3c36669 into master Apr 12, 2021
@lanceleonard lanceleonard deleted the jira-doc-566 branch April 12, 2021 21:53
lanceleonard added a commit that referenced this pull request Apr 13, 2021
* First set of quick fixes.

* More edits and updates, mainly to redirects.

* Updating python library to be more current.
lanceleonard added a commit that referenced this pull request Apr 13, 2021
* Follow-up to PR #1231; fixing assorted minor issues  (#1311)

* First set of quick fixes.

* More edits and updates, mainly to redirects.

* Updating python library to be more current.

* Updating backup article aliases (#1315)

* First set of quick fixes.

* More edits and updates, mainly to redirects.

* Updating python library to be more current.

* Updating aliases

* Initial draft for release notes
lanceleonard added a commit that referenced this pull request Apr 13, 2021
* DOC-390 Add data eviction to Active-Active database creation procedure (#1171)

* deprecation of featureset and protocol version 0

part of 6.0.20.
Was mentioned on RN of 6.0.12 and will also be part of 6.0.20 RN

* Update upgrading.md (#1267)

* Doc 390 eviction for active active (#1268)

* DOC-525 Migrate to Active-Active with Replica Of (#1208)

* no-AA-eviction-RoF.md (#1281)

* Doc-89 priority mTLS changes for 6.0.20 (#1293)

* security changes
* changes to mTLS cert add process

* Doc-646 TLS revision (#1296)

* Jira DOC-306: Document LDAP support in new release (#1289)

* Initial check-in.  Need images and edit.

* Updating links to location for cluster ldap doc.

* Adding screenshots and performing initial edit pass.

* Additional edits, links, and minor tweaks.

* Changes for editorial feedback.

* Fixing broken link

* Fixing tpyo in broken link fix

* Fixing tpyo in the typo fix for the broken link fix.

* Update cluster-based-ldap-authentication.md

* More feedback updates.

Co-authored-by: Kyle Banker <kyle.banker@redislabs.com>
Co-authored-by: Alon Magrafta <50071256+AlonMagrafta@users.noreply.github.com>

* Initial check-in of Azure SAS support notes; also includes high level edits for new style guidelines. (#1301)

* cipher suite options for cluster config (#1314)

* DOC-392 A/A support for bitfield (#1300)

* RS v6.0.20 release notes (#1316)

* Follow-up to PR #1231; fixing assorted minor issues  (#1311)

* First set of quick fixes.

* More edits and updates, mainly to redirects.

* Updating python library to be more current.

* Updating backup article aliases (#1315)

* First set of quick fixes.

* More edits and updates, mainly to redirects.

* Updating python library to be more current.

* Updating aliases

* Initial draft for release notes

Co-authored-by: Ben Mansheim <github@mansheim.com>
Co-authored-by: Alon Magrafta <50071256+AlonMagrafta@users.noreply.github.com>
Co-authored-by: Kaitlyn Michael <76962844+kaitlynmichaelredislabs@users.noreply.github.com>
Co-authored-by: Kyle Banker <kyle.banker@redislabs.com>
Co-authored-by: Kaitlyn Michael <76962844+kaitlynmichael@users.noreply.github.com>
Co-authored-by: kaitlyn.michael <kaitlyn.michael@redislabs.com>
Co-authored-by: Kyle Banker <banker@users.noreply.github.com>
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