-
Notifications
You must be signed in to change notification settings - Fork 142
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
Making critical components robust #599
Comments
Working on it, to setup a basic automated test suite for critical components. |
Sounds good like a good idea to me. |
100% agree |
@YashKumarVerma very good initiative. I have some idea for UI components. Current UI folder is using very basic vanilla js setup and it will be difficult to add more complex functionality in app going further. I can convert this into a next.js setup with cleaner UI components and APIs . This will help the project to add more functionality faster in the UI . @DavidAJohnston let me know if I can proceed with this ? |
Have added added a test suite that is checking things of the backend |
@DavidAJohnston have added a PR, requesting a look at it |
@YashKumarVerma should this issue be closed? |
Why
Impact
The text was updated successfully, but these errors were encountered: