Skip to content

Conversation

@jackkeller
Copy link
Collaborator

This pull request adds a new settings page and several settings controls to the project. The settings page includes a blank layout and the necessary routing. The settings controls include boolean, color, file, number, and string controls, each with their own stories for testing. This PR also includes some styling updates and fixes a bug related to rendering a multiline textarea.

Testing Locally

  1. Log in using ADMIN or SUPER account
  2. In the web-ui directory, run yarn start to view http://localhost:5173/admin/settings
  3. Your process.env.NODE_ENV must be set to development to view the settings page with sample controls
  4. You can run yarn ladle if you want to see the Storybook examples for the settings controls

Once verified that this is working as expected locally, the sample controls in the settings page can be replaced with the actual settings controls. And that can happen on this PR or when the settings are ready to be added to the project.

Copy link
Contributor

@vhscom vhscom left a comment

Choose a reason for hiding this comment

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

Approved

@jackkeller jackkeller force-pushed the feature-2356/add-application-settings-page branch from 9cd769f to a8d5827 Compare May 20, 2024 20:54
@jackkeller jackkeller merged commit 2153f0d into develop May 20, 2024
@jackkeller jackkeller deleted the feature-2356/add-application-settings-page branch May 20, 2024 21:07
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.

4 participants