Skip to content
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

fix(contributors): sort contributors alphabetically #316

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

NikGor
Copy link
Contributor

@NikGor NikGor commented Aug 15, 2023

Resolved issue #298 regarding sorting and filtering of contributors in the dropdown list.

Resolved issue Hexlet#298 regarding sorting and filtering of contributors in the dropdown list.
@NikGor
Copy link
Contributor Author

NikGor commented Aug 15, 2023

issue вроде как неактуально и перекрывается другим issue, но список по алфавиту отсортировал.
В отсортированном списке стала видна новая проблема - множество значений None

@fey
Copy link
Collaborator

fey commented Aug 16, 2023

с None можно посмотреть, откуда они берутся. Можно в настройках пользователя указать отображаемый ник. Либо сортировать список пользователей.

@fey fey merged commit c477278 into Hexlet:main Aug 16, 2023
1 check failed
@fey
Copy link
Collaborator

fey commented Aug 16, 2023

@NikGor пайплайн упал. но код я уже смержил. Поправьте, пожалуйста и пришлите новый ПР

@NikGor
Copy link
Contributor Author

NikGor commented Aug 16, 2023

@fey извиняюсь, не обратил внимания что строчка стала слишком длинной. Исправил, отправил ПР

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