-
Notifications
You must be signed in to change notification settings - Fork 77
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
[DONE] Fix event info display #1234
base: develop
Are you sure you want to change the base?
Conversation
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.
Merci Céline :)
Juste une remarque sur l'affichage du proprio
<div> | ||
<a class="btn btn-link btn-sm" href="{% url 'videos:videos' %}?owner={{ event.owner.get_username }}" {% if request.GET.is_iframe %}target="_blank"{% endif %}> | ||
<span class="pod-meta-title">{% trans 'Added by:' %}</span> | ||
<a class="btn btn-link btn-sm px-0" href="{% url 'videos:videos' %}?owner={{ event.owner.get_username }}" {% if request.GET.is_iframe %}target="_blank" {% endif %} title="{% blocktrans with video_user_name=video.owner.get_username %}Show video list of user “{{ video_user_name }}”{% endblocktrans %}" data-bs-toggle="tooltip" data-bs-placement="bottom"> |
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.
utilise plutot event.owner
sans le getUsername, car owner se charge d'afficher le username ou le fullname en fonction des parametres voulus
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.
Merci Céline. A y être, si tu peux supprimer le get_username, cela serait parfait (au moins, le username ne s'affichera que selon le paramétrage HIDE_USERNAME, que j'ai découvert récemment :) )
Loïc, tu voudrais que j'ôte la méthode get_username de l'objet User ? c'est ça ? parce qu'elle est encore appelée à plein d'endroits |
Non surtout pas ;) |
Reprend l'affichage des infos de l'évènement à l'image des infos d'une vidéo et affiche le max de viewer d'un live calculé par le heartbeat et stocké en base