Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Patient list should sort by given name ascending by deafult #1967

Open
jackcmeyer opened this issue Apr 5, 2020 · 5 comments
Open

Patient list should sort by given name ascending by deafult #1967

jackcmeyer opened this issue Apr 5, 2020 · 5 comments
Assignees
@akshay-ap
Labels
馃殌enhancement an issue/pull request that adds a feature to the application in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Projects
Version 2.0
Milestone
v2.0

Comments

@jackcmeyer
Copy link
Member

jackcmeyer commented Apr 5, 2020 •
edited

馃殌 Feature Proposal

By default, patients should be sorted by given name ascending.

Motivation

The list should not be sorted in a random way.

Blocked by

#1969
@jackcmeyer jackcmeyer self-assigned this Apr 5, 2020
@jackcmeyer jackcmeyer transferred this issue from HospitalRun/hospitalrun Apr 5, 2020
@jackcmeyer jackcmeyer added core-team indicates that the issue will be completed by a core team member 馃殌enhancement an issue/pull request that adds a feature to the application patients issue/pull request that interacts with patients module labels Apr 5, 2020
@jackcmeyer jackcmeyer added this to the v2.0 milestone Apr 5, 2020
@jackcmeyer jackcmeyer removed the core-team indicates that the issue will be completed by a core team member label Apr 5, 2020
@jackcmeyer jackcmeyer removed their assignment Apr 5, 2020
@jackcmeyer jackcmeyer added the 馃毀blocked issue cannot be worked on for some reason label Apr 5, 2020
@jackcmeyer jackcmeyer added help wanted indicates that an issue is open for contributions and removed 馃毀blocked issue cannot be worked on for some reason labels Apr 18, 2020
@sujithnath
Copy link

@jackcmeyer Can I take it?

akshay-ap added a commit to akshay-ap/hospitalrun-frontend that referenced this issue May 3, 2020
@akshay-ap
feat(viewpatients): add a new field 'index', paging in next direction
d1c55e7 
New 'index' field is a combination of 'fullName' and 'code' so that uniqueness is guaranteed and
paging logic can relay on this field. A new field in PageRequest to store a reference field for
getting next page.

feat HospitalRun#1969, feat HospitalRun#1967
@vicdurai
Copy link

vicdurai commented May 27, 2020 •
edited

Can I pickup this issue or is someone working on this already?

@akshay-ap
Copy link
Contributor

FYI: The patient list will be sorted by name in #2026.

@vicdurai
Copy link

Cool since this is being worked on, I will not work on this. @tehkapa Please can you unassign me from this issue and assign it to @akshay-ap since he already has a PR for it.

@matteovivona
Copy link
Contributor

Ok. Done! Sorry @akshay-ap

@matteovivona matteovivona assigned akshay-ap and unassigned vicdurai May 27, 2020
@matteovivona matteovivona added the in progress indicates that issue/pull request is currently being worked on label May 27, 2020
@akshay-ap akshay-ap mentioned this issue May 31, 2020
Merged
@ghost ghost removed the help wanted indicates that an issue is open for contributions label Jun 14, 2020
@ghost ghost moved this from To do to Done in Version 2.0 Jun 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@akshay-ap akshay-ap

Labels
馃殌enhancement an issue/pull request that adds a feature to the application in progress indicates that issue/pull request is currently being worked on patients issue/pull request that interacts with patients module
Projects
Version 2.0
  
Done
Milestone
v2.0
Development

No branches or pull requests
5 participants
@matteovivona @sujithnath @vicdurai @akshay-ap @jackcmeyer