Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

Close access to the taxonomy pages #617

Closed
wants to merge 3 commits into from

Conversation

yasik78
Copy link
Contributor

@yasik78 yasik78 commented Jun 25, 2017

#378

Steps for review

  • Login as admin

  • Check term admin/structure/taxonomy/manage/area/overview - page 404

  • Check term admin/structure/taxonomy/manage/color/overview - page 404

  • Check term admin/structure/taxonomy/manage/facility_type/overview - page 404

  • Check term admin/structure/taxonomy/manage/blog_category/overview - ok

Update from ymcatwincities/openy
Close access to the taxonomy pages ymcatwincities#378
@yasik78 yasik78 added the PR: Needs Review Needs someone review ( code ) label Jun 25, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 25, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 25, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 25, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 27, 2017
@Sanchiz
Copy link
Contributor

Sanchiz commented Jun 27, 2017

@yasik78

  1. We should close access only to followign taxonomy terms
    image

  2. Please use module rabbit_hole to close access. It should be already in the distribution.

@Sanchiz Sanchiz added PR: Needs Work Unfinished task. Issues still there and removed PR: Needs Review Needs someone review ( code ) labels Jun 27, 2017
@Sanchiz Sanchiz modified the milestone: Version 1.3 Jun 27, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 29, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 29, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 29, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 29, 2017
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 29, 2017
@ddrozdik
Copy link
Contributor

Also we need to add upgrade path here

@Sanchiz Sanchiz modified the milestones: Version 1.3, Version 1.4 Jul 6, 2017
@AlexNetman AlexNetman marked this as a duplicate of #669 Jul 14, 2017
@Sanchiz
Copy link
Contributor

Sanchiz commented Jul 26, 2017

This has been fixed by another community member here #669. Thank you for contribution!

@Sanchiz Sanchiz closed this Jul 26, 2017
@Sanchiz Sanchiz removed this from the Version 1.4 milestone Jul 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: Needs Work Unfinished task. Issues still there
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants