Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chi created an issue. See original summary.

Chi’s picture

Issue summary: View changes
Neslee Canil Pinto’s picture

Status: Active » Needs work
Ramya Balasubramanian’s picture

Assigned: Unassigned » Ramya Balasubramanian
Ramya Balasubramanian’s picture

Hi @chi, @Neslee Canil Pinto,

I have updated the patch and please have a look.

Ramya Balasubramanian’s picture

Ramya Balasubramanian’s picture

Assigned: Ramya Balasubramanian » Unassigned
pradeepjha’s picture

Status: Needs review » Needs work
FileSize
29.43 KB

@Ramya Balasubramanian path is not apply. Plz try attaching patch again.

Ramya Balasubramanian’s picture

Assigned: Unassigned » Ramya Balasubramanian
Ramya Balasubramanian’s picture

Status: Needs work » Needs review
FileSize
281 bytes

Hi @pradeepjha,

Please have a look at this updated patch.

Status: Needs review » Needs work
Ramya Balasubramanian’s picture

Updating the patch again.

Status: Needs review » Needs work

The last submitted patch, 12: permission-issue-3133908-12.patch, failed testing. View results

Ramya Balasubramanian’s picture

Status: Needs work » Needs review
Ramya Balasubramanian’s picture

Assigned: Ramya Balasubramanian » Unassigned
rishab.singh’s picture

Assigned: Unassigned » rishab.singh
Status: Needs review » Reviewed & tested by the community
FileSize
14.89 KB

Hi @Ramya Balasubramanian,

I have tested this patch without any issue and it looks good to me
Attached screenshot for reference and moving this ticket to RTBC.

rishab.singh’s picture

Assigned: rishab.singh » Unassigned

Neslee Canil Pinto’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.