-
Notifications
You must be signed in to change notification settings - Fork 0
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
enhance people view #113
base: develop
Are you sure you want to change the base?
enhance people view #113
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -107,8 +107,7 @@ const HorizontalLettersMenu = ({ linkedLetters }) => { | |||
} | |||
|
|||
/* | |||
* people are coming into this component from | |||
* fetchStrapiPeople with this shape: | |||
* people are coming into this component with this shape: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we also filter the people array by the active field? currently it's pulling in both active and inactive staff members for the staff page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ooh yes. thanks for catching that!
lib/dashboard/people.js
Outdated
return p.firstName.localeCompare(q.firstName) ? p.lastName.localeCompare(q.lastName) : -1 | ||
}); | ||
|
||
const ood = people.filter(person => person.teams.some(t => t.name === 'Office of the Director')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use dashboard uid or slug
b45785e
to
eda256e
Compare
this PR aims to list people on the the
/people
view alphabetically. a couple related efforts are folded in here: