Replace Github with GitHub in assets/js/wins.js updateOverlay

[daras-cu]

Overview

We need to replace instances of Github and github with GitHub so that we display the company name correctly, and test the changes to ensure the behavior and appearance of any and all related webpages are unchanged.

Action Items

• Open assets/js/wins.js in your IDE
• In function updateOverlay(i) change

  			makeIcon(data[i][github_url], overlayIcons, 'github-icon', '/assets/images/wins-page/icon-github-small.svg', 'Github profile for ' + data[i][name]);

to

  			makeIcon(data[i][github_url], overlayIcons, 'github-icon', '/assets/images/wins-page/icon-github-small.svg', 'GitHub profile for ' + data[i][name]);

• Find all the files that are connected to this change. (e.g., if its a yml or md file, what files or scripts use the data stored in that file). And document briefly how they work together.
• Define where you expect the change to show up on the website (e.g., the footer of the website). Can be on multiple pages
• Write a test procedure, in a comment, and then use it to test the change. Include a link to this comment in your PR.
• Add/commit changes then create a PR as usual. Document the test procedure in the Pull Request.

Resources

• This issue resulted from ER: Find all the places that GitHub should be GitHub and fix #6161

Merge Team

• After closing this issue, check off the dependency in ER: Find all the places that GitHub should be GitHub and fix #6161. If all dependencies have been completed, close ER: Find all the places that GitHub should be GitHub and fix #6161

[HackforLABot]

Hello @dvernon5, we appreciate you taking on this issue, however it looks like you're already working on another issue at this time. Please wait until your current issue is merged before taking on another issue. :)

We are going to unassign you from this issue so you can focus on your current issue.

Hfla appreciates you! :)