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

Enhancement: Refactor layercontrol #527

Merged
merged 28 commits into from
Dec 21, 2023

Conversation

srijitcoder
Copy link
Collaborator

@srijitcoder srijitcoder commented Dec 15, 2023

Important Changes

  • Segregated all the functions/methods into separate files to improve readability.
  • Component-level methods are now organized within /src/methods/*/*.js
  • Introduced enums for centralizing and managing constants.
  • Stories have been segregated into distinct files and consolidated into a single file using index.js.
  • Portions of the code were rewritten to enhance readability and reduce complexity.
  • Extensive comments have been added for better comprehension.
  • Test cases are now separated into individual files and consolidated using index.js.
  • Efforts were made to ensure that component-level code remains under 180 lines and functions under 50 lines.

@srijitcoder srijitcoder added enhancement New feature or request layercontrol labels Dec 15, 2023
@srijitcoder srijitcoder marked this pull request as ready for review December 19, 2023 09:14
@silvester-pari silvester-pari merged commit 9968efb into main Dec 21, 2023
3 checks passed
@silvester-pari silvester-pari deleted the layercontrol/enhancement/refactor-code branch December 21, 2023 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request layercontrol
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants