[rmamet]

What

This adds caching to the Major Project Form to save form input, in case the website happens to reload

Why

I was assigned to #598, which wants to fix this

Test Plan

I manually checked by hosting conditional on my personal computer

Env Vars

I did not touch any environment variables

Documentation

The documentation was not updated

Checklist

• Tested all changes locally

[datadog-computersciencehouse]

⚠️ Warnings

Your PR has warnings. Please review the issues below.

🚦 1 Pipeline job failed

Sonarqube | Sonarqube Analysis

View in Datadog · View in GitHub Actions

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 1d29c3f | Docs | View more details | Give us feedback!}

[BigSpaceships]

this is great, just formatting things (i gave up, i'd check all the indentations)

[BigSpaceships]

btw you should update the branch

[NickJ-7010]

I have two additional comments:

1. The skills get added to the real skill list but also to another list above it. When you click to remove the skill from the real list it doesn't delete nor does it properly restore on refresh. Items that are present in the real list are duplicated upon reload on the other list which represents the localStorage state.
2. You should add debounce functionality so that it doesn't try to recompute the localStorage on every keypress but possibly after a second or two, should be easy to do with a setTimeout and clearTimeout

[NickJ-7010]

Suggested change

	addSkill(skill) {
	let input = document.getElementById("skill-input")
	addSkill(skill) {
	if (skill === "")
	return;
	let input = document.getElementById("skill-input");

This removes errors / duplicate event handlers that happen when you click on the skills field without typing anything. Errors occur when there are no skills in the list and duplicate event handlers get added to the last skill element in the list