Skip to content

Order clients by surnames #880

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.

Sign up for GitHub

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

Jump to bottom
Open
giacy86 wants to merge 3 commits into
base: development
Choose a base branch
from
Open

Order clients by surnames #880

giacy86 wants to merge 3 commits into from

Conversation

@giacy86
Copy link
Contributor

@giacy86 giacy86 commented Nov 4, 2022

Pull Request Checklist

  • My code follows the code formatting guidelines.
  • I have an issue ID for this pull request.
  • I selected the corresponding branch.
  • I have rebased my changes on top of the corresponding branch.

Issue Type (Please check one or more)

  • Bugfix
  • Improvement of an existing Feature
  • New Feature

@giacy86
Copy link
Contributor Author

giacy86 commented Nov 4, 2022 •
edited by nielsdrost7
Loading

Addresses issue #879

@giacy86 giacy86 mentioned this pull request Nov 4, 2022
Closed
@nielsdrost7 nielsdrost7 changed the base branch from master to development November 4, 2022 17:32
@fabm3n
Copy link

fabm3n commented Sep 27, 2023

Hi @giacy86 ,

i don't think it will be a good idea to just change the default behaviour, because the surname is an optional field and therefore can be empty.
It would be better to make the client overview more flexible to order all columns like everyone likes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@giacy86 @fabm3n