remove people.jsx, refactor people.scss

[Pomax]

We're no longer using the people page, but some of its SCSS is used by other pages/components, so we should pull those two things apart, remove the people page code itself, and make sure we relocate the still-used styling.

[HSR05]

Is this still open?

[Pomax]

It is, but this one is not actually as easy as the original comment makes it sound, so we'll be tackling this one internally.

[mmmavis]

Just a note that /source/sass/views/fellowships/people.scss (which its code is identical to /source/js/components/people/people.scss) is not being imported and used. We should decide which file we wanna keep and remove the not in-use one.

/cc @youriwims

[youriwims]

I'm not sure if /source/js/components/people/people.scss is still being used...would have to look more into that. But, we can remove /source/js/components/people/people.scss. I think I was going to update that one, but then found /source/sass/wagtail/profiles.scss and settled on updating that file and forgot to remove the other. Feel free to create a ticket for it and assign me 😄

[youriwims]

Update: Looks like the entire source/js/components/people directory which includespeople.jsx, people.scss, and person.jsx can safely be removed. Styles in people.scss are no longer in use since the new Fellowship Directory was built ( Ref #2511, #2685, #2474 ). Lastly, the Person module will be obsolete once all of the fellowship related files and people.jsx are removed.

[mmmavis]

Awesome. Don't forget to remove people_page.html and other related bits! 👍

[youriwims]

Thanks! 😸

[youriwims]

Temporarily blocked