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

database creation #755

Closed
wants to merge 2 commits into from
Closed

Conversation

sarthakjalan05
Copy link
Contributor

@sarthakjalan05 sarthakjalan05 commented Oct 6, 2024

What GitHub issue does this PR apply to?

Resolves #495

What changed and why?

To publish a stats page of visits, as well as track missing data in your project, you can implement a logging or analytics system.

(If editing website code) Please add screenshots

Checklist

  • I claimed any associated issue(s) and they are not someone else's
  • I have looked at documentation to ensure I made any revisions correctly
  • I tested my changes locally to ensure they work
  • For language files, I have validated the edited files are valid JSON and data shows up correctly
  • For website code edits, I have added or edited any appropriate unit tests for my changes

Any additional comments or things to be aware of while reviewing?

@geekygirlsarah
Copy link
Member

Hey @sarthakjalan05, #495 is for actually publishing an actual stats page. You made a temp page and some JS for it that looks like it might set some sample things up, but it's not done and that page isn't actually linked anywhere. Also the database tables already exist (tables web_lookupdata and web_sitevisit, see web/migrations/0001_initial.py for the file that sets it up) so no SQL files are necessary for this.

Go look around again and let me know if you need any help or if maybe you don't want to work on this issue anymore.

@sarthakjalan05
Copy link
Contributor Author

@geekygirlsarah I tried to work on that but couldnt find a solution

@sarthakjalan05 sarthakjalan05 deleted the analysis branch October 8, 2024 08:07
@geekygirlsarah
Copy link
Member

@geekygirlsarah I tried to work on that but couldnt find a solution

Do you want me to try to explain more, or would you rather not work on the issue?

@sarthakjalan05
Copy link
Contributor Author

@geekygirlsarah I tried to work on that but couldnt find a solution

Do you want me to try to explain more, or would you rather not work on the issue?
@geekygirlsarah i would rather not work because I tried to solve but couldn't find a solution. Thanks for considering my help

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.

[Idea] Publish stats page of visits of looked up data and/or missing data
2 participants