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
Change public upload, fixes INFRA-205 #343
Conversation
@timoludwig @lukasboehm97 do you have time for a quick review? Otherwise I'd deploy the change myself. As the influence on the rest of the functionality is rather limited that should be acceptable. |
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.
Thanks a lot 👍
In general it seems to work, I just found a problem when the discipline is changed multiple times.
Also, I noticed that the logo image path is broken on the current develop branch, could you add {% load static %}
to the top of the template file and replace this line:
<link rel="icon" href="/docs/4.0/assets/img/favicons/favicon.ico"> |
with
<link rel="icon" href="{% static 'images/logo.svg' %}">
and this line:
<img class="mb-4" src="static/images/logo.png" alt="" width="72" height="72"> |
with
<img class="mb-4" src="{% static 'images/logo.svg' %}" alt="" width="72" height="72">
as well as add a changelog entry for this?
8c892d1
to
173d33a
Compare
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.
Thanks! 👍
Maybe add the following line to the changelog?
* [ [#343](https://github.com/digitalfabrik/lunes-cms/issues/343) ] Remove module dropdown in public upload
* Directly show documents when selecting a module. * Fix static files.
173d33a
to
f201829
Compare
Definitely. I did not notice that there now is a changelog :D |
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.
🎉
Short description
As we now have many photos in our content management system, the list of missing images per training set is rather short. This makes it cumbersome to go through all disciplines and training sets to find and upload missing images.
Proposed changes
When selecting a discipline, we now directly show associated documents. This skips the step of selecting a training set.
Resolved issues
Fixes: INFRA-205