maciejpi/nordabiz
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(messages): correct avatar path (/static/ not /static/uploads/), use last_active_at for presence, add last_active_at to members API
Some checks are pending
NordaBiz Tests / Unit & Integration Tests (push) Waiting to run
Details
NordaBiz Tests / E2E Tests (Playwright) (push) Blocked by required conditions
Details
NordaBiz Tests / Smoke Tests (Production) (push) Blocked by required conditions
Details
NordaBiz Tests / Send Failure Notification (push) Blocked by required conditions
Details

Browse Source 
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Maciej Pienczyn 2026-03-27 14:44:31 +01:00
parent f6b9b9154f
commit 131e6f0118
2 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
blueprints/messages/conversation_routes.py
View File

@ -88,7 +88,7 @@ def _conversation_list_json(db, membership):
u = other_members[0].user
display_name = u.name or u.email.split('@')[0]
if u.avatar_path:
avatar_url = '/static/uploads/' + u.avatar_path
avatar_url = '/static/' + u.avatar_path
# Online status for 1:1 conversations
is_online = False
@ -202,7 +202,7 @@ def conversations_page():
'id': current_user.id,
'name': current_user.name or current_user.email.split('@')[0],
'email': current_user.email,
'avatar_url': ('/static/uploads/' + current_user.avatar_path) if current_user.avatar_path else None,
'avatar_url': ('/static/' + current_user.avatar_path) if current_user.avatar_path else None,
}
return render_template(
@ -394,7 +394,7 @@ def api_conversation_detail(conv_id):
avatar_url = None
if m.user.avatar_path:
avatar_url = '/static/uploads/' + m.user.avatar_path
avatar_url = '/static/' + m.user.avatar_path
members_list.append({
'user_id': m.user_id,
'name': m.user.name or m.user.email.split('@')[0],
@ -403,6 +403,7 @@ def api_conversation_detail(conv_id):
'company_name': company_name,
'role': m.role,
'last_read_at': m.last_read_at.isoformat() if m.last_read_at else None,
'last_active_at': m.user.last_active_at.isoformat() if m.user.last_active_at else (m.user.last_login.isoformat() if m.user.last_login else None),
'is_online': is_user_online(m.user_id),
})

8
static/js/conversations.js
View File

@ -389,8 +389,8 @@
if (other && other.is_online) {
subtitle.innerHTML = '<span class="online-status-dot"></span> online';
subtitle.classList.add('is-online');
} else if (other && other.last_read_at) {
subtitle.textContent = formatPresence(other.last_read_at);
} else if (other && (other.last_active_at || other.last_read_at)) {
subtitle.textContent = formatPresence(other.last_active_at || other.last_read_at);
} else {
subtitle.textContent = '';
}
@ -1477,8 +1477,8 @@
if (other && other.is_online) {
subtitle.innerHTML = '<span class="online-status-dot"></span> online';
subtitle.classList.add('is-online');
} else if (other && other.last_read_at) {
subtitle.textContent = formatPresence(other.last_read_at);
} else if (other && (other.last_active_at || other.last_read_at)) {
subtitle.textContent = formatPresence(other.last_active_at || other.last_read_at);
}
}
}