ER: Remove Extraneous `$` in `wins.js`

[dvernon5]

Emergent Requirement - Problem

An invalid $ character is present in the wins.js file on line 31, causing a potential issue with the wins page functionality.

Issue you discovered this emergent requirement in

• Google Apps Script - Add Unique IDs to Wins entries #8121

Date discovered

05/18/2025

Did you have to do something temporarily

• YES
• NO

Who was involved

@daras-cu

What happens if this is not addressed

Selecting I became part of a caring community on the Share your wins with us form, will cause the wins page to crash, disrupting the user experience.

Resources

• wins.js file: website/assets/js/wins.js
• Data file: website/_data/external/_wins-data.json
• Icon file: website/assets/images/wins-page/wins-badges/community.svg
• demo video by @daras-cu showing wins page user information update: video demo

Recommended Action Items

• Make a new issue
• Discuss with team
• Let a Team Lead know

Potential solutions [draft]

1. In website/assets/js/wins.js on line 31, update $community.svg to community.svg by removing the $ character.
2. Develop a test procedure to verify that the wins page renders correctly when "I became part of a caring community." is selected in the "What do you want to celebrate (select all that apply)? field in website/_data/external/_wins-data.json.
   • The community.svg icon (website/assets/images/wins-page/wins-badges/community.svg) should display under the developer’s summary section if "I became part of a caring community." is included in their responses.
   • Hint: You may manually edit website/_data/external/_wins-data.json on a separate test branch to simulate a selection on the "Share your wins with us" form.
3. In your PR, include an explanation of the testing you performed and relevant screenshots.

[github-actions]

Hi @dvernon5.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing:

• Complexity, Role, Feature

NOTE: Please ignore this comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards.
To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

• Wiki: How to create issues
• Wiki: How to read and interpret labels

[daras-cu]

Hi @dvernon5, thanks for writing this issue. Let's make a couple changes, then it should be good to go:

• Make the title more concise, something like "Remove extraneous $ in wins.js"
• Under the draft solution, let's get rid of the example test case. As this will be a medium issue, the developer should figure out how to test independently. You could include a "hint" that _wins-data.json can be manually edited to simulate a selection on the Wins form.