-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update functions.js #87
Conversation
Disable the multilingual library (temporarily)
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.
Please remove any commended code. Commends on this case provide no value so they should be deleted.
We need to keep the code. |
Otherwise, we need an option for the wordpress site "is_multilingual". (This is better) |
Version control systems is for that reason, git helps keep track of old changes without commending code in the codebase for historical reasons. Multilanguage is a great feature that`s been on "issues" section here #7 |
Add option for multilingual wordpress site.
Take into account the config option for wordpress
@@ -49,6 +49,7 @@ const OWASP_CONFIG = { | |||
|
|||
const WORDPRESS_CREDENTIALS = { | |||
url: 'https://www.iee.ihu.gr', | |||
is_multilingual: false, |
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.
The code style convection we follow is camel cased, so please try follow this for consistency.
In this case "is_multilingual" shloud be "isMultilingual"
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.
Please try following the code style for naming variables, functions etc.
Other than that, the PR looks fine.
Do whatever changes on code you wish!!!!!! |
Disable the multilingual library (temporarily)