professor-l / classic-tetris-project

A Discord and twitch bot for Classic Tetris Monthly
MIT License
23 stars 23 forks source link

Add django-wiki to support permission controlled wiki articles #82

Closed Fireworks closed 1 year ago

Fireworks commented 1 year ago

I added this package because I think it would fit the requirements we need to support a permission-controlled Wiki for CTM ops information.

I think some extra configuration may be needed, but was curious to get this up so people can play around with it.

This will create new DB tables and adds a new set of routes for the wiki under "/wiki" - though I had some trouble figuring out why it was complaining about slug conflicts, so I disabled the check for it (https://github.com/professor-l/classic-tetris-project/compare/wiki-site?expand=1#diff-59dcbaacfb65ddb37a8c5544c085c426b5a6c5aa816d280200bfbb489b69a02bR234). I believe it has something to do with how the routing file is setup since we aren't using regex matchers. Seems to work fine with disabling this check.

I am wondering about the changes to the requirements file though, it seems like it added a number of additional dependencies... but perhaps they are all requisites of this new library.

Let me know if you guys have any concerns or questions!

Django-wiki: https://github.com/django-wiki/django-wiki

CTM_wiki