search

mgaitan / waliki

A wiki engine powered by Django and Git
http://waliki.pythonanywhere.com
BSD 3-Clause "New" or "Revised" License
309 stars 56 forks source link

Move page form doesn't show the current slug #137

Closed wagnerflo closed 7 years ago

wagnerflo commented 7 years ago

What's the motivation for not showing the current slug, but instead an empty field on the MovePageForm (achieved by exclude = ('slug',))?

Doesn't that just make it more tedious to use? If I just want to fix a typo in the slug I'll have to at a minimum copy & paste it from the URL...

wagnerflo commented 7 years ago

After some more digging through the code I now understand the reason. I've amended the commit to just provide initial={...} to the MovePageForm constructor instead of removing the exclude.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 71.104% when pulling 9edbe0c5eda9aa0717f5c3dde473726491cd3c8f on wagnerflo:master into 07306662181a8259bef63b39bb9a830c24a38283 on mgaitan:master.