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

Usage statistics: Azure JS method #569

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Usage statistics: Azure JS method #569

wants to merge 1 commit into from

Conversation

joeflack4
Copy link
Member

@joeflack4 joeflack4 commented Sep 19, 2023

Updates

Added azure monitoring as per Martin's instructions: https://teams.microsoft.com/l/message/19:[email protected]/1695137986622?context=%7B%22contextType%22%3A%22chat%22%7D

Notes

@Sigfried I branched this off of the last successful deployment commit: 1cc2688ba421edba3de11c0736c4625165701029

I was able to successfully redeploy from temporary branch deployment-2023-09-07-branch, before branching off and creating this usage-statistics branch.

I also kept a tag for this. I do that sometimes for deployments: https://github.com/jhu-bids/TermHub/tree/deployment-2023-09-07

Related

@joeflack4 joeflack4 added the new feature New feature or request label Sep 19, 2023
@joeflack4 joeflack4 changed the title Usage statistics Usage statistics: Azure JS method Sep 20, 2023
@joeflack4
Copy link
Member Author

@Sigfried Leaving this one to you now. We could merge this code into develop, but we currently dan't deploy develop due to #570. So if you want to do any more investigation / tweaking / redeploying of this, maybe stick to this branch for now.

@joeflack4
Copy link
Member Author

Left this off with:

  1. Martin looking into some things. Some React module might help.
  2. How to utilize: We also need to be more familiar with the logs and the visualizations to understand what we can get out of it.
  3. Can we see when they hit an error? That's most important.

@joeflack4
Copy link
Member Author

I'm going to keep this open, but not sure this is still relevant. We may want to close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
Status: 2. Soon
Development

Successfully merging this pull request may close these issues.

2 participants