vincbeck commented on code in PR #32217:
URL: https://github.com/apache/airflow/pull/32217#discussion_r1245597878
##########
airflow/www/templates/appbuilder/navbar_right.html:
##########
@@ -66,12 +66,9 @@
{% if not current_user.is_anonymous %}
<li class="dropdown">
<a class="dropdown-toggle" href="#">
- <span class="navbar-user-icon" title="{{g.user.get_full_name()}}">
- {% if current_user.first_name and current_user.last_name %}
- <span>{{ (current_user.first_name[0] +
current_user.last_name[0]).upper() }}</span>
- {% else %}
- <span class="material-icons">person</span>
- {% endif %}
+ <span class="navbar-user-icon" title="{{ auth_manager.get_user_name()
}}">
+ {% set user_names = auth_manager.get_user_name().split(" ", 1) %}
+ <span>{% for name in user_names %}{{ name[0].upper() }}{% endfor
%}</span>
Review Comment:
I know :( But that's the only fair solution I could find to make it backward
compatible. And I dont think the change is breaking though, this is just an
icon on the top right corner
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
