Skip to content

[RFR] Fix disabling views#1336

Merged
Kmaschta merged 3 commits intomasterfrom
fix_disabling_views
Apr 5, 2017
Merged

[RFR] Fix disabling views#1336
Kmaschta merged 3 commits intomasterfrom
fix_disabling_views

Conversation

@floo51
Copy link
Copy Markdown
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 force-pushed the fix_disabling_views branch from e007fa9 to 453cb69 Compare April 5, 2017 09:44
@floo51 floo51 changed the title Fix disabling views [RFR] Fix disabling views Apr 5, 2017
@floo51
Copy link
Copy Markdown
Contributor Author

floo51 commented Apr 5, 2017

Mergeable - @Kmaschta

.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
Copy Markdown
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
Copy Markdown
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
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 this pull request may close these issues.

2 participants