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

When I Add New User, And Use The User to view dashboard ,it has an error #2005

Closed
kira8565 opened this Issue Mar 31, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@kira8565

kira8565 commented Mar 31, 2016

Problem description

Steps to reproduce the problem

  1. Add New User And assign dashboard to the user
  2. change current user to the new user which add before
  3. click the dashboard
  4. can not view the dashboard which i assign before,and occur an error like this
Unhandled rejection TypeError: Cannot read property 'set' of undefined
    at ProxyClass._getDashboardActions (webpack:///./src/components/dashboard/Dashboard.jsx?:120:70)
    at ProxyClass.proxiedMethod (webpack:///./~/react-proxy/modules/createPrototypeProxy.js?:44:30)
    at ProxyClass.eval [as _getDashboardActions] (webpack:///./~/react-hot-api/modules/makeAssimilatePrototype.js?:15:37)
    at ProxyClass.render (webpack:///./src/components/dashboard/Dashboard.jsx?:154:14)
    at ProxyClass.proxiedMethod (webpack:///./~/react-proxy/modules/createPrototypeProxy.js?:44:30)
    at ProxyClass.eval [as render] (webpack:///./~/react-hot-api/modules/makeAssimilatePrototype.js?:15:37)
    at ReactCompositeComponentMixin._renderValidatedComponentWithoutOwnerOrContext (http://localhost:8080/build/vendor.js:38634:35)
    at ReactCompositeComponentMixin._renderValidatedComponent (http://localhost:8080/build/vendor.js:38654:33)
    at wrapper [as _renderValidatedComponent] (http://localhost:8080/build/vendor.js:5007:22)
    at ReactCompositeComponentMixin.mountComponent (http://localhost:8080/build/vendor.js:38267:31)
    at wrapper [as mountComponent] (http://localhost:8080/build/vendor.js:5007:22)
    at Object.ReactReconciler.mountComponent (http://localhost:8080/build/vendor.js:6191:36)
    at ReactDOMComponent.ReactMultiChild.Mixin.mountChildren (http://localhost:8080/build/vendor.js:41719:45)
    at ReactDOMComponent.Mixin._createContentMarkup (http://localhost:8080/build/vendor.js:39393:33)
    at ReactDOMComponent.Mixin.mountComponent (http://localhost:8080/build/vendor.js:39281:30)
    at Object.ReactReconciler.mountComponent (http://localhost:8080/build/vendor.js:6191:36)

and I Change This code ins dashboard.jsx

<MenuItem href={ApiRoutes.StartpageController.set('dashboard', this.props.dashboard.id).url}>

to

<MenuItem href={StartpageStore.set('dashboard', this.props.dashboard.id).url}>

it work

Environment

  • Graylog Version: Graylog-Master (Beta2)
  • Elasticsearch Version: 2.1.1
  • MongoDB Version: 3.2
  • Operating System: Mac
  • Browser version: Chorm 48

@edmundoa edmundoa added bug web labels Mar 31, 2016

@edmundoa edmundoa added this to the 2.0.0 milestone Mar 31, 2016

@edmundoa

This comment has been minimized.

Member

edmundoa commented Mar 31, 2016

Hi,

Thank you for reporting this issue. ApiRoutes.StartpageController is indeed undefined, although the suggested workaround doesn't really work (the page will load, but the button doesn't work as intented). Will fix it for the next beta.

@edmundoa edmundoa self-assigned this Mar 31, 2016

@edmundoa edmundoa closed this in f938cdd Mar 31, 2016

@edmundoa

This comment has been minimized.

Member

edmundoa commented Mar 31, 2016

This issue has been fixed in f938cdd. Unfortunately there is another issue that makes impossible for regular users to reset their startpage. There is a PR already for that, so this should work as expected once #1987 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment