Fix Languages /Technologies Filters for Projects 4587

[t-will-gillis]

Fixes #4587

What changes did you make and why did you make them ?

• Edited current-projects.js to fix bug that was causing the "Languages / Technologies" filter to show incorrect results when projects do not have "Technologies" listed.
• Specifically, edited Line 483 of the file so that the ternary operator defaults to the project.languages.map(lang => lang) list if "project.technologies" is not present.
• Changes also include a tab indent on function projectCardComponent(project)

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

Visuals after changes are applied

[github-actions]

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b t-will-gillis-fix-languages-technologies-filters-4587 gh-pages
git pull https://github.com/t-will-gillis/website.git fix-languages-technologies-filters-4587

[StephenTheDev1001]

Availability: M-F 10am - 12pm
ETA: Wed, May 17th

[Jaretzbalba]

Availability: 2 hours
ETA: 5/16 EOD

[Jaretzbalba]

@t-will-gillis This looks really great! I see that the new code allows the filter function to work effectively regardless of the technologies attribute being present. Appreciate your hard work on this issue. Thanks!

[mjshelton12]

Availability: Tue-Wed 9am-1pm, Fri 9am-4pm
ETA: Wed by end of day

[mjshelton12]

First off, great work on the basics. Your branches are correct, you linked to an issue, you provided a thorough explanation of what you did/why you did it, and included appropriate visuals. Your code is clean and effective, plus it shows perfectly in chrome desktop, tablet, and phone as well as safari desktop. Excellent work!

I was able to get the expected result both with "Write for All" and "BallotNav". I tried to search a bunch of different options to see if anything unexpected came up and all seemed normal.

One problem I noticed that is outside the scope of this issue and is NOT caused by this PR as it's the same behavior as the live site is that if you check a technology that is used and then one that isn't, you get the same results as if you didn't check the tech not used. i.e if I check off CSS, BallotNav pops up then if I check Python (which it doesn't use), BallotNav still hangs around. Just wanted that noted here as it is NOT a bug caused by this fix. Because that is the case, I'm approving this request.

[t-will-gillis]

@Jaretzbalba and @mjshelton12 - thanks for the reviews and comments! Good observation about the effects of multiple filters- i.e. selecting 'Python' and 'CSS' returns any projects that have either / or condition, rather than both / all conditions. There is another set of issues (#4525 and #4427) coming soon that ultimately will be adding a 'tools' filter alongside the others. When we start working on that feature we probably will want to revisit how we want the filters to behave