-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Syntax changes and model update changes #1056
Conversation
{% for incident in officer.incidents | sort(attribute='date') | reverse %} | ||
{% for incident in officer.incidents %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now covered in the view.
if "created_by" in data: | ||
link.created_by = parse_int(data.get("created_by")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The created_by
field shouldn't be updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this also be replaced with a constant?
https://github.com/lucyparsons/OpenOversight/pull/1056/files#diff-0ea02af7be8529b5f1eb21efd1af13f1d633060c50ee2529a1064ddba1616c93L76-L80
@sea-kelp, at the very least it should be an f-string. We can change that on the next code change. |
Description of Changes
A lot of small syntax changes along with adding missing field inclusions.
Tests and linting
develop
branch.pytest
passes on my local development environment.pre-commit
passes on my local development environment.