search

bihealth / sodar-core

SODAR Core: A Django-based framework for building scientific data management web apps
MIT License
9 stars 1 forks source link

Crash in addremotesite command calling TimelineAPI.add_event() #1425

Closed mikkonie closed 3 months ago

mikkonie commented 3 months ago

It seems the addremotesite command crashes when calling TimelineAPI.add_event(). The user kwarg is not correctly provided.

Furthermore, it seems the description kwarg is badly formed.

I guess this command hasn't seen much use as this has gone unreported for some time. It might be prudent to consider removing it since we also have a REST API view for this, but for now it's just faster to fix it and carry on.

Also one of the reasons this hasn't been caught is that we're missing tests for the command, see #352.

mikkonie commented 3 months ago

Fixed.