Conversation
@@ -460,8 +460,28 @@ class GenericApplicantContainer extends React.Component { | |||
} | |||
}); | |||
const filteredApplicants = this.filterApplicants(checkedApplicants); | |||
const fields = ['firstName', 'lastName', 'email', 'id', 'tags']; | |||
const fields = ['firstName', 'lastName', 'email', 'id', 'tags.accepted', 'tags.abc']; |
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.
Can we set fields to an empty array?
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.
we could, i think it's better if we keep firstName, lastName, email, and id so it keeps it nicely ordered, what do you think?
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.
That's a good point, I'm fine with keeping firstName
, lastName
, email
, and id
, but can we get rid of tags.accepted
, tags.abc
?
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.
yup sounds good
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.
LGTM, great work!
ποΈ Ticket(s): Closes #
π· Changes
Exporting selected applicants should include all fields and tags of each applicant
π Notes
π¦ Testing Instructions
Explain how to test your changes, if applicable.