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

Fix navigation in LDAP components #3670

Merged
merged 1 commit into from Mar 29, 2017
Merged

Fix navigation in LDAP components #3670

merged 1 commit into from Mar 29, 2017

Conversation

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Mar 29, 2017

Fixes #3485, backport from #3651
(cherry picked from commit 1fafbf1)

* Fix navigation between ldap config and groups

As there is no easy way to access each component through an URL any
longer, use callbacks to let the parent decide which component to
render.

* Remove unused config prop

* Use jsx extension for ldap component

* Fix error handling in ldap groups component

- Do not try to render an object, as that doesn't work
- Display no configuration message when server returns 400, as that only
  happens when LDAP is not correctly configured. That message is a bit
  more clear and less scary

Fixes #3485
(cherry picked from commit 1fafbf1)
@edmundoa edmundoa added this to the 2.2.3 milestone Mar 29, 2017
@joschi joschi self-assigned this Mar 29, 2017
@joschi
joschi approved these changes Mar 29, 2017
Copy link
Contributor

@joschi joschi left a comment

LGTM. 👍

@joschi joschi merged commit 2cc20be into 2.2 Mar 29, 2017
4 checks passed
4 checks passed
ci-web-linter Jenkins build graylog-pr-linter-check 1481 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details
@joschi joschi deleted the issue-3485-2.2 branch Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.