Skip to content

Stats: не показывать группы пользователя без названия#666

Merged
DedSec256 merged 4 commits intomasterfrom
groups-fix-user-groups
Apr 22, 2026
Merged

Stats: не показывать группы пользователя без названия#666
DedSec256 merged 4 commits intomasterfrom
groups-fix-user-groups

Conversation

@semrosin
Copy link
Copy Markdown
Contributor

Была проблема с отображением запятых, если студент состоит в группах без названия:
image

Теперь ненужные запятые не отображаются

@semrosin semrosin requested a review from DedSec256 April 22, 2026 17:21
<Stack direction="row" spacing={1}>
<GroupIcon style={{fontSize: "12px"}}/>
<div>{studentGroups
.filter(g => g.name && g.name.length != 0)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.filter(g => g.name && g.name.length != 0)
.filter(g => g.name?.trim())

@DedSec256 DedSec256 merged commit fe096c8 into master Apr 22, 2026
1 check passed
@DedSec256 DedSec256 deleted the groups-fix-user-groups branch April 22, 2026 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants