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

Upstash Redis Dashboard Rework #2434

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

fahreddinozcan
Copy link

@fahreddinozcan fahreddinozcan commented Jul 5, 2024

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

Anything else we should know when reviewing?

@fahreddinozcan fahreddinozcan requested a review from a team as a code owner July 5, 2024 08:40
@fahreddinozcan fahreddinozcan marked this pull request as draft July 5, 2024 08:40
@emarsha94
Copy link
Collaborator

@fahreddinozcan Is this a feature request for Redis to update their dashboard?

@fahreddinozcan
Copy link
Author

Hey,

we've internally updated the structure of the Upstash dashboard and added few more charts etc. This is not finalized yet, but I'll ask for your review when it's finalized.

@fahreddinozcan fahreddinozcan marked this pull request as ready for review July 26, 2024 08:44
@fahreddinozcan
Copy link
Author

Hey @JoshPatel13, this is finalized now. Open for review

upstash/metadata.csv Outdated Show resolved Hide resolved
@buggyhunter
Copy link

@emarsha94 @JoshPatel13 Can we merge the PR if all is OK?

@fahreddinozcan
Copy link
Author

@emarsha94 @JoshPatel13 I guess it's now ready?

@bgoldberg122
Copy link
Contributor

bgoldberg122 commented Sep 11, 2024

Hi @fahreddinozcan 👋 I just want to confirm that you want to delete the original dashboard rather than keep both. Once resolved I can merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants