Nice work so far. I did find one problem: import re is missing from the file so the new re.match line throws an error.
And one way it could be made a bit better would be to add 'error' as a second parameter to the flash() call so that it shows up in red instead of green. If you look through the file you can see some other flash() calls that do that for example.
Nice work so far. I did find one problem:
import re
is missing from the file so the newre.match
line throws an error.And one way it could be made a bit better would be to add
'error'
as a second parameter to theflash()
call so that it shows up in red instead of green. If you look through the file you can see some otherflash()
calls that do that for example.Can you make those changes and push them? Thanks!
Ideally code changes would have a tests for them too, but this is a very simple change and testing a flash message has a few tricks. But maybe next contribution you make can have a test too :) https://forge-allura.apache.org/docs/development/contributing.html#testing
Fixed!!
https://forge-allura.apache.org/p/allura/git/merge-requests/24/
Fixed!!
https://forge-allura.apache.org/p/allura/git/merge-requests/24/