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

Issue 701: Add stats for Internet Protocol version #895

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sebbo94BY
Copy link

@Sebbo94BY Sebbo94BY commented Jul 9, 2023

  • Adds a new database column
  • Updates the SQL statements to save the new information in the database
  • Adds a new donut graph for "IPv4 vs IPv6"

Draft / In progress, because:

  • Untested
  • The SELECT SQL statement and/or regular calc job for this new statistic is missing

Solves #701

- Adds a new database column
- Updates the SQL statements to save the new information in the database
- Adds a new donut graph for "IPv4 vs IPv6"
@Sebbo94BY
Copy link
Author

@Newcomer1989 I believe, only the following things are missing to get this working and testable:

  • Adjust the JavaScript code in libs/statsindex.js to fill the donut graph with values. (I've already added something, but I didn't understood your code logic here yet.)
  • A regular job to calculate the statistic? Thus, an additional new table/column? 🤔

Can you please support me with the missing changes since you know your code much better than anyone else? Thank you!

We can also have a quick talk on a TeamSpeak server. Either in English or German. :)

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.

1 participant