CommentFileSizeAuthor
#8 adding-404-template-2.patch1.81 KBsd9121
#4 adding-404-template.patch1.83 KBsd9121
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sreenivas Bttv created an issue. See original summary.

hansa11’s picture

hansa11 CreditAttribution: hansa11 as a volunteer and at QED42 commented
Assigned: Unassigned » hansa11
sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
Assigned: hansa11 » sd9121
sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
FileSize
adding-404-template.patch1.83 KB

Added template for 404 page and styling for this.

sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
Assigned: sd9121 » Unassigned
Status: Active » Needs review
Sreenivas Bttv’s picture

Sreenivas Bttv CreditAttribution: Sreenivas Bttv at QED42 commented
Status: Needs review » Needs work

@sd9121, #4 Patch failing because of conflicts in the .theme file.

sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
Assigned: Unassigned » sd9121
sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
FileSize
adding-404-template-2.patch1.81 KB

@Sreenivas Bttv, I have fixed merge conflict. Please review my patch.
Thanks !

sd9121’s picture

sd9121 CreditAttribution: sd9121 at QED42 commented
Assigned: sd9121 » Unassigned
Status: Needs work » Needs review
Sreenivas Bttv’s picture

Sreenivas Bttv CreditAttribution: Sreenivas Bttv at QED42 commented
Status: Needs review » Reviewed & tested by the community

@sd9121,
Reviewed #8 patch and working without any issue.

Sreenivas Bttv’s picture

Sreenivas Bttv CreditAttribution: Sreenivas Bttv at QED42 commented
Status: Reviewed & tested by the community » Fixed

@sd9121,
Thanks for your efforts
Patch committed and pushed to 8.x-1.x branch.

Sreenivas Bttv’s picture

Sreenivas Bttv CreditAttribution: Sreenivas Bttv at QED42 commented
Status: Fixed » Closed (fixed)