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.

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

Order clients by surnames #880

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

Order clients by surnames #880

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

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.

None yet
2 participants
@giacy86 @fabm3n