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

OWA-41: Add manage settings page #7

Merged
merged 1 commit into from
Sep 8, 2017
Merged

OWA-41: Add manage settings page #7

merged 1 commit into from
Sep 8, 2017

Conversation

jomadoye
Copy link
Contributor

@jomadoye jomadoye commented Sep 7, 2017

JIRA TICKET NAME:

OWA-41: Create Add-on Management setting UI as a migrate from the Legacy UI

SUMMARY:

This PR creates a page for the manages OWA settings.

Copy link
Contributor

@wanjikum wanjikum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work Jedidah!

@dkayiwa
Copy link
Member

dkayiwa commented Sep 7, 2017

Thanks @wanjikum for reviewing. 👍

bower.json Outdated
"Web",
"App"
],
"homepage": "https://github.com/andela-jomadoye/openmrs-owa-addonmanager",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the above url correct?

event.preventDefault();
this.setState({
AppBaseURLValue: '',
AppFolderPathValue: 'appdata\\owa',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the backward slashes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dkayiwa It is only 1 backslash that appears, the second backslash is used to escape it. The final output is appdata\owa

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not use a forward slash?


componentWillMount() {
this.setState({
AppFolderPathValue: 'appdata\\owa',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the backward slashes?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The double backslash is used to escape the backslash slash.

@jomadoye
Copy link
Contributor Author

jomadoye commented Sep 8, 2017

Thanks @dkayiwa all issues have been fixed.

@dkayiwa dkayiwa merged commit 698a575 into openmrs:master Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants