Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New LWC-based Config Management #1436

Merged
merged 64 commits into from
Mar 23, 2024
Merged

New LWC-based Config Management #1436

merged 64 commits into from
Mar 23, 2024

Conversation

aheber
Copy link
Contributor

@aheber aheber commented Mar 19, 2024

This creates the new LWC wizard for making or editing DLRS configurations

JimBTek and others added 30 commits January 10, 2024 10:29
Improve tab behavior and restructure data so it is easier to work with.
Enable correct icon per object (delayed loading)
Enable correct loading and clearing of values from parent
Improve search perf and sorting
octatau and others added 24 commits February 14, 2024 01:07
add open source cRonstrue project, dependencies, and license statements
Copy link
Member

@sfenton3 sfenton3 left a comment

Choose a reason for hiding this comment

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

This still needs:

  • Test classes/coverage
  • Remove VF/Controller for old New Wizard
  • QA

sfenton3
sfenton3 previously approved these changes Mar 22, 2024
Copy link
Member

@sfenton3 sfenton3 left a comment

Choose a reason for hiding this comment

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

This is a pretty safe change and does not change the core functionality. Merging this will allow the team to easily beta test and QA. We need to make sure to add additional test coverage before cutting a new release.

@aheber aheber merged commit a49b617 into main Mar 23, 2024
3 checks passed
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.

7 participants