openstudyroom icon indicating copy to clipboard operation
openstudyroom copied to clipboard

confusion with both senses of primary leagues

Open angelsesma opened this issue 4 years ago • 2 comments

/ladder subdirectory to be included in the href string

angelsesma avatar Jan 07 '21 00:01 angelsesma

Thanks for spotting that. Actually the issue is more of an undocumented and unused feature:

One league is called primary. There is a button in league admin tools to set it: image

The url /league/ will always direct to the primary league.

So the issue here is the primary league have not been updated for ages.

Mind that the name is quite confusing since it also refer to primary leagues where player are automatically enrolled. image

I think a better fix would be to control the url /league/. I see 2 ways:

  1. always redirect it to /ladder/
  2. Admins should update the One primary league everymonth.

I guess 1 is better.

Change would happen here: https://github.com/climu/openstudyroom/blob/0ae71e90fe36766d002d221b63ace9f18d517e45/league/views.py#L263-L264

with much inspiration from: https://github.com/climu/openstudyroom/blob/0ae71e90fe36766d002d221b63ace9f18d517e45/league/views.py#L307-L317

climu avatar Jan 07 '21 09:01 climu

thanks for the heads up! will give it a shot :)

angelsesma avatar Jan 07 '21 13:01 angelsesma