search

wincowgerDEV / OpenSpecy-package

Analyze, Process, Identify, and Share, Raman and (FT)IR Spectra
http://wincowger.com/OpenSpecy-package/
Creative Commons Attribution 4.0 International
27 stars 11 forks source link

Log fix #57

Closed wincowgerDEV closed 3 years ago

wincowgerDEV commented 3 years ago

Improve logging of events in shiny.

wincowgerDEV commented 3 years ago

@zsteinmetz I figured out how to get the logging working with a combination of mongolite and loggit. Loggit will work if the app is on the desktop and mongo will work if on shinyapps. They store the data as basically carbon copies for cross compatibility. With this update, we can remove google analytics which I think will make it so that we don't have any third party cookies on the website. Need to double check that cookies aren't getting created somewhere else though. The logging is also now dependant on the user selection of "share data" so that they don't have to get logged if they don't want to.

One thing we need to still do is make sure that the loggit file is going to the tempdir or a dir that the user wants it to go to. Perhaps we can make that compatible with the location that the user specifies where their library files or maybe we should make that a unique location?

zsteinmetz commented 3 years ago

Awesome work! This will also fix wincowgerDEV/OpenSpecy-shiny#9 then.

One thing we need to still do is make sure that the loggit file is going to the tempdir or a dir that the user wants it to go to. Perhaps we can make that compatible with the location that the user specifies where their library files or maybe we should make that a unique location?

I can take a look at this if you like. Should we already merge this branch into wincowgerDEV/OpenSpecy-package#54 or do you like to add some more stuff?

zsteinmetz commented 3 years ago

I think we're set.

wincowgerDEV commented 3 years ago

@zsteinmetz great work! I agree. I think we are ready.