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

Refactor JavaScript in the application #25

Open
4 tasks
kallewesterling opened this issue Jul 12, 2024 · 1 comment
Open
4 tasks

Refactor JavaScript in the application #25

kallewesterling opened this issue Jul 12, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@kallewesterling
Copy link
Member

kallewesterling commented Jul 12, 2024

Description

Currently, all the JavaScript in the Seshat project is spread across multiple .js files and some embedded JavaScript within the HTML. This issue aims to refactor the JavaScript code, ensure it is well-documented, and potentially implement JavaScript testing.

Objectives

Tasks

  • Consolidation of all code:
    • Identify and list all JavaScript code currently embedded in HTML files.
    • Create or update .js files to include the identified JavaScript code.
    • Ensure all HTML files are updated to reference the external .js files instead of embedded scripts.
  • Refactoring:
    • Refactor the consolidated JavaScript code to improve readability, maintainability, and performance.
    • Use consistent coding standards and practices throughout the codebase.
    • Ensure proper modularisation of the JavaScript code.
  • Documentation:
    • Add detailed comments and documentation to all JavaScript functions and modules.
    • Include information about the purpose, parameters, return values, and any important notes for each function.
    • Ensure documentation is clear and easy to understand.
  • Testing:
    • Identify and set up a suitable JavaScript testing framework (e.g., Jest, Mocha, Jasmine -- see list here).
    • Write unit tests for all critical functions and modules to verify their behaviour.
    • Integrate tests into the project's CI/CD pipeline to ensure ongoing code quality.

Definition of Done

  • All JavaScript code is consolidated into appropriate .js files.
  • No JavaScript is embedded directly within HTML files.
  • The code is fully documented, following the project's documentation standards.
  • Unit tests are implemented, and all tests pass successfully.
  • The refactored code is reviewed, merged into the main branch, and deployed to the QA environment.
  • The changes are demonstrated to the team and accepted by the product owner.
@edwardchalstrey1
Copy link

@kallewesterling if you have worked on this want to open a draft PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants