#927-Default-Layout-options-endpoint #1118

Merged

a22erigr merged 4 commits from #927-Default-Layout-options-endpoint into team_2_week_7

2026-05-25 09:11:30 +00:00

c24elipe commented

2026-05-25 04:48:32 +00:00

Collaborator

Added refresh token## Environment

This was developed on an Arch-based Linux distribution.

What has been done?

Before I break down what I have done, I didn't really know exactly what was wanted, so I just implemented the bullet points—they were:

A default,
A way to load it.

I think that is what I have done. Please tell me if I have misunderstood the assignment, and I will fix it if so be the case.

frontend/src/pages/dashboard/Dashboard.js
- Removed unused import.
frontend/src/pages/dashboard/utils/SaveGrid.js
- Added a function to set the layout to the default.
- Also set the default to be the one to be loaded first if no "getFromLocalStorage" value is set.
frontend/src/pages/dashboard/utils/presets/DefaultLayout.json
- This is just a basic layout example.

Issue

#927

# Added refresh token## Environment This was developed on an Arch-based Linux distribution. ## What has been done? Before I break down what I have done, I didn't really know exactly what was wanted, so I just implemented the bullet points—they were: 1. A default, 2. A way to load it. I think that is what I have done. Please tell me if I have misunderstood the assignment, and I will fix it if so be the case. * **frontend/src/pages/dashboard/Dashboard.js** * Removed unused import. * **frontend/src/pages/dashboard/utils/SaveGrid.js** * Added a function to set the layout to the default. * Also set the default to be the one to be loaded first if no "getFromLocalStorage" value is set. * **frontend/src/pages/dashboard/utils/presets/DefaultLayout.json** * This is just a basic layout example. ## Issue #927

c24elipe added the

needs review

label

2026-05-25 04:48:32 +00:00

c24elipe added 3 commits

2026-05-25 04:48:33 +00:00

Added Defualt layout 180639e255

Fixed spelling mistake 12240e764c

Added Example of how use the defult layout 3c374b9c11

c24elipe referenced this pull request

2026-05-25 04:49:03 +00:00

Default Layout options endpoint #927

c24elipe added 1 commit

2026-05-25 04:50:16 +00:00

Removed reset to defaultLayoutData 44a946faa0

a22erigr requested review from c22abdal

2026-05-25 08:19:22 +00:00

c22abdal approved these changes

2026-05-25 09:09:41 +00:00

c22abdal left a comment

Collaborator

Review

The implementation works as expected, and the default layout is correctly loaded when no saved layout exists in local storage. The added reset/default functionality also works well, and the cleanup of unused imports is appreciated.

One thing that may need clarification is the PR title in the description mentioning refresh tokens, since the actual changes are related to dashboard layout presets and default loading behavior. It may be good to rename or update the PR description to better reflect the implemented functionality.

# Review The implementation works as expected, and the default layout is correctly loaded when no saved layout exists in local storage. The added reset/default functionality also works well, and the cleanup of unused imports is appreciated. One thing that may need clarification is the PR title in the description mentioning refresh tokens, since the actual changes are related to dashboard layout presets and default loading behavior. It may be good to rename or update the PR description to better reflect the implemented functionality.