-
Notifications
You must be signed in to change notification settings - Fork 1k
Description
The template at https://github.com/pypi/warehouse/blob/main/warehouse/admin/templates/admin/projects/release_detail.html is missing the new Dynamic
field.
Additionally, Provides-Extra
is commented out for some reason, and should be uncommented:
warehouse/warehouse/admin/templates/admin/projects/release_detail.html
Lines 311 to 320 in f87e97d
{# | |
<tr> | |
<td>Provides-Extra</td> | |
<td> | |
{% for extra in release.provides_extras %} | |
<code>{{ extra }}</code><br> | |
{% endfor %} | |
</td> | |
</tr> | |
#} |
Ideally, this template would use the enum from #13606 instead of manually enumerating all these fields. This is probably to complicated for a "Good First Issue" because it requires knowing which fields are lists.
Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide.
If you are working on this issue and have questions, feel free to ask them here, in the #pypa-dev
chat channel on Libera, the PyPA Discord or on the Discourse.
Screenshot Required: If your pull request makes a visual change, include a screenshot of your update. This helps our team give you feedback faster.