Variables should be passed to templates through context. E.g. in projects-id-metadata-create.html we access request.GET.step directly. I'm just thinking that this pattern cannot be mirrored in the command-line interface and it also makes the dependencies of / input to templates more opaque, so we might as well avoid it.
We should only pass variables to a template (through context) that are actually used in that template. E.g. in render_projects_id_metadata_create we pass table_name to context, but I wasn’t able to find where it is used in the corresponding template.
context
. E.g. inprojects-id-metadata-create.html
we accessrequest.GET.step
directly. I'm just thinking that this pattern cannot be mirrored in the command-line interface and it also makes the dependencies of / input to templates more opaque, so we might as well avoid it.context
) that are actually used in that template. E.g. inrender_projects_id_metadata_create
we passtable_name
tocontext
, but I wasn’t able to find where it is used in the corresponding template.