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

CreationView should redirect to ShowView when EditionView is disabled #393

Closed
ramSeraph opened this issue Apr 10, 2015 · 5 comments · Fixed by #400
Closed

CreationView should redirect to ShowView when EditionView is disabled #393

ramSeraph opened this issue Apr 10, 2015 · 5 comments · Fixed by #400
Assignees

Comments

@ramSeraph
Copy link

CreationView upon success redirects to EditionView, but if EditionView is disabled this goes nowhere.

In this case it should try and redirect to ShowView or ListView which ever is still available

@fzaninotto
Copy link
Member

Let me rephrase this: For a particular entity, you want to authorize creation, but not edition. Could you share the use case that leads to this weird decision?

@ramSeraph
Copy link
Author

There is only one field on the entity which is editable and even that is just a toggle. I would prefer if that can just be toggled by a custom button in the list view( this I have already implemented ).

Creation on the other hand has a lot more fields and does call for a separate view.

Even otherwise, there will definitely be use cases where someone might want to just create and delete and not edit an entity. Automatically redirecting to an empty edit view might be undesired in these cases as well.

@fzaninotto
Copy link
Member

Again, can you explain me the domain leading to such decision?

@ramSeraph
Copy link
Author

Ah, you meant the domain, It is the admin page for a social app and the entities are related to the invite codes for the same.

@ramSeraph
Copy link
Author

Thanks for the quick turnaround. Checked and it works

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.

2 participants