Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[RFR] Fix disabling views #1336

Merged
merged 3 commits into from
Apr 5, 2017
Merged

[RFR] Fix disabling views #1336

merged 3 commits into from
Apr 5, 2017

Conversation

floo51
Copy link
Contributor

@floo51 floo51 commented Apr 5, 2017

It should fix the disabling of views which wasn't working when you had fields initialized inside the view.

It also fix the test fixtures which were making tags a read only entity and trying to go to its edit page anyway.

@floo51 floo51 changed the title Fix disabling views [RFR] Fix disabling views Apr 5, 2017
@floo51
Copy link
Contributor Author

floo51 commented Apr 5, 2017

Mergeable - @Kmaschta

@@ -84,8 +84,7 @@
.baseApiUrl('http://localhost:3000/') // The base API endpoint can be customized by entity
.identifier(nga.field('id')); // you can optionally customize the identifier used in the api ('id' by default)

var tag = nga.entity('tags')
.readOnly(); // a readOnly entity has disabled creation, edition, and deletion views
Copy link
Contributor

Choose a reason for hiding this comment

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

What are the effects of your admin-config update on readOnly feature?

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 effect is that it now works

@Kmaschta Kmaschta merged commit cd64165 into master Apr 5, 2017
@Kmaschta Kmaschta deleted the fix_disabling_views branch April 5, 2017 11:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants