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

web/database: add usage indicator #1771

Merged
merged 6 commits into from
Nov 24, 2024

Conversation

SoggySaussages
Copy link
Contributor

@SoggySaussages SoggySaussages commented Nov 22, 2024

Adds a meter of database usage and info to the database page. Meter turns orange when 75% full and red when 90% full. There’s also a web alert at 95%.

database-usage

Signed-off-by: SoggySaussages [email protected]

youtube/feed.go Outdated Show resolved Hide resolved
youtube/web.go Outdated Show resolved Hide resolved
customcommands/assets/customcommands-database.html Outdated Show resolved Hide resolved
<h4>{{.TotalDatabaseUsage}} Entries used of total {{.TotalDatabaseCapacity}}</h4>
<p>
Total calculated by member count multiplied by 50{{if .IsGuildPremium}}0{{end}} (<code>{{.ActiveGuild.MemberCount}}*50{{if .IsGuildPremium}}0{{end}}</code>).{{if not .IsGuildPremium}} Get 10 times the
database space with <a href="/premium">premium</a>!{{end}}
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm torn on plugging premium here—ultimately Des's call to make, but personally I'm not a big fan of this on the front-end.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's plugged for the custom commands check, I think it's beneficial to know that its possible to increase the size.

customcommands/web.go Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

Stray change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hotfixes on master but not on dev I believe

@SoggySaussages
Copy link
Contributor Author

database-usage

@ashishjh-bst
Copy link
Contributor

please rebase this to dev

@SoggySaussages
Copy link
Contributor Author

That's done.

@ashishjh-bst ashishjh-bst merged commit 26493e0 into botlabs-gg:dev Nov 24, 2024
1 check passed
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