- https://codeberg.org/kiara
- Joined on
2025-02-04
looks like this is shown on either side of the conditional? if so, could we take it out, to save on duplication for this line, and maybe even get rid of the else here?
the 0/1 feels a bit harder to read (without having seen views.py) - would it be possible to maybe pass the info using a dictionary to make it more clear what this does?
this feels a bit fragile in case the exact wording changes - would we have a nicer more robust to do this?
maybe add a comma at the end of the line as well? technically python does not require this, but this makes it so adding an extra line at the end won't require changing the old line, making diffs a little easier to review.
thanks for the quick pr! i added a few nitpicks, but glad to see this working already. ✌️😺