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

Dashboard list links lead to "Page Not Found" #423

Closed
stephenkubovic opened this issue May 8, 2015 · 5 comments · Fixed by #434
Closed

Dashboard list links lead to "Page Not Found" #423

stephenkubovic opened this issue May 8, 2015 · 5 comments · Fixed by #434

Comments

@stephenkubovic
Copy link

Clicking an entity list link in the dashboard results in a "Page Not Found". This looks to be due to a trailing slash in the goToList() function within the maDashboardPanel directive. Click "Recent posts", "Last comments", or "Recent tags" in the demo app to observe this behaviour.

Relevant code:

https://github.com/marmelab/ng-admin/blob/master/src/javascripts/ng-admin/Main/component/directive/maDashboardPanel.js#L21

scope.gotoList = function () {
    $location.path(scope.entity().name() + '/list/');
};
@dekhaus
Copy link

dekhaus commented May 29, 2015

I have version 0.7.0 of ng-admin and I'm still experiencing this bug - aka the trailing '/'. How do I get the fix/patch ?

Thanks
Dave

@fzaninotto
Copy link
Member

You need to use the master branch, we haven't released a new version for it.

@dekhaus
Copy link

dekhaus commented Jun 14, 2015

Hi

I just upgraded to the latest 'master' version and the above links issue is fixed. Thanks.

I do have an additional question .... in the prior release, if I didn't explicitly configure a Dashboard page - I got a page with a link for every managed resource. Using the current release - how can I replicate this feature of the previous release ?

Thanks
Dave

@fzaninotto
Copy link
Member

@dekhaus I'm in the process of revamping the dashboard API. You'll have a populated dashboard by default, with the ability to customize everything (resizing panels, adding your owns, etc). Watch the pull request notifications for updates.

@dekhaus
Copy link

dekhaus commented Jun 15, 2015

Thanks

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 a pull request may close this issue.

3 participants