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

improve wording for database sharding options in UI #13543

Merged
merged 2 commits into from
Feb 12, 2021

Conversation

jsteemann
Copy link
Contributor

@jsteemann jsteemann commented Feb 11, 2021

Scope & Purpose

Improve the wording for sharding options displayed in the web interface.

Instead of offering flexible and single, now use the more intuitive Sharded and OneShard options, and update the help text for them (refer to attached screenshots for details).

We may need to adjust documentation referring to "single" and "flexible" before we can merge this. We also need to decide whether this should be backported to 3.7 or not.

  • 💩 Bugfix (requires CHANGELOG entry)
  • 🍕 New feature (requires CHANGELOG entry, feature documentation and release notes)
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification
  • 📖 CHANGELOG entry made

Backports:

  • Backports required for: 3.7 (no PR yet)

Testing & Verification

  • This change is a trivial rework / code cleanup without any test coverage.
  • The behavior in this PR was manually tested

Link to Jenkins PR run:
http://172.16.10.101:8080/view/PR/job/arangodb-matrix-pr/13971/

Screenshot from 2021-02-11 20-27-12
Screenshot from 2021-02-11 20-25-39

@jsteemann jsteemann added this to the devel milestone Feb 11, 2021
@jsteemann jsteemann requested a review from hkernbach February 11, 2021 19:37
@jsteemann jsteemann marked this pull request as ready for review February 12, 2021 13:29
@jsteemann jsteemann merged commit 1cd5b32 into devel Feb 12, 2021
@jsteemann jsteemann deleted the bug-fix/improve-sharding-wording-in-ui branch February 12, 2021 13:30
elfringham pushed a commit to elfringham/arangodb that referenced this pull request Apr 20, 2021
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