ondemand icon indicating copy to clipboard operation
ondemand copied to clipboard

Remove analytics.lua

Open TheGamer1002 opened this issue 1 year ago • 1 comments

Fixes #2808 This should be safe in theory, as my IDE claims there were no usages in code.

TheGamer1002 avatar Jun 20 '24 23:06 TheGamer1002

This is going to be a fairly large change so I'm not sure if it's indeed a good first issue.

First, we need to remove the references here (this is the bit that calls it): https://github.com/OSC/ondemand/blob/c3e71188d8a254db51b01e070fadcbce2ecdd37b/ood-portal-generator/templates/ood-portal.conf.erb#L307-L311

That's likely to break a bunch of test cases.

Then we should likely remove the internal variable: https://github.com/OSC/ondemand/blob/c3e71188d8a254db51b01e070fadcbce2ecdd37b/ood-portal-generator/lib/ood_portal_generator/view.rb#L69-L70

Lastly removing the config option from the example config file https://github.com/OSC/ondemand/blob/c3e71188d8a254db51b01e070fadcbce2ecdd37b/ood-portal-generator/share/ood_portal_example.yml#L194-L202

johrstrom avatar Jun 24 '24 13:06 johrstrom

Hey, I'm going to close this as it still needs work and some time has gone by.

Hope everything's going well and summer institute was worth your time!

johrstrom avatar Aug 28 '24 16:08 johrstrom