search

uclibs / uc_drc

DRC migration planning and work
Other
0 stars 0 forks source link

Add sidekiq dashboard for admin #90

Closed crowesn closed 3 years ago

crowesn commented 3 years ago

Fixes #86 Requires #89

Add route for sidekiq dashboard, limited to user with admin role.

scherztc commented 3 years ago

@crowesn : Do we need the sidekiq.yml in the config folder? Do we need any of the adapter stuff in the .env files?

Here is the sidekiq commit for ucrate: https://github.com/uclibs/ucrate/commit/345f7afcea045122bc1edfb3abc81a3c7470572a

Just double check that we don't need any of the other configuration while we are here.

scherztc commented 3 years ago

@crowesn : Maybe we add the config file later when we add BulkRAX. It seems a little heavy to test the existence of a yml file or a sidekiq queue.

'''    yaml_object = YAML.load(content)      expect(yaml_object.length).to be > 0 '''

or remove the yml and re-submit

crowesn commented 3 years ago

I looks like Coveralls decided it wanted better coverage on User model - I'll throw a test in and see if that does anything. https://coveralls.io/builds/34504811/source?filename=app%2Fmodels%2Fuser.rb