-
-
Notifications
You must be signed in to change notification settings - Fork 256
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
Fix: Demo Model Page #903
Fix: Demo Model Page #903
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a set of failing unit tests at tests/unit/views/demo/selectDemoModel.spec.js, if we could fix these up
I'll fix this |
@jgadsden can i have your assistance in how to carry forward with the changes required in the tests/unit/views/demo/selectDemoModel.spec.js. |
made changes in selectDemoMode.spec.js
Merge pull request #1 from OWASP/Fixes/DemoModelPage
@jgadsden i was trying to update the unit test cases but not getting the exact solution. It'll be great if i get your assistance in this. |
revert the changes in unit test
Summary:
Description for the changelog:
Embedded the i18n with the Demo Model Page For better User Experience.
Other info:
Thanks for submitting a pull request!
Please make sure you follow our code_of_conduct.md and our contributing guidelines contributing.md