Skip to content

Add localStorage saving to majorProjectForm - #624

Open
rmamet wants to merge 5 commits into
ComputerScienceHouse:developfrom
rmamet:major-form-cache
Open

rmamet wants to merge 5 commits into
ComputerScienceHouse:developfrom
rmamet:major-form-cache

Conversation

@rmamet

@rmamet rmamet commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

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

datadog-computersciencehouse Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Pipelines

⚠️ 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: 4120627 | Docs | View more details | Give us feedback!

@BigSpaceships BigSpaceships left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread frontend/javascript/modules/majorProjectForm.js Outdated
Comment thread frontend/javascript/modules/majorProjectForm.js Outdated
Comment thread frontend/javascript/modules/majorProjectForm.js Outdated
Comment thread frontend/javascript/modules/majorProjectForm.js
@BigSpaceships

Copy link
Copy Markdown
Contributor

btw you should update the branch

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants