Theme has some dependencies due to which it is not installed in the website. So update the README file with all dependencies.

CommentFileSizeAuthor
#13 2979283-12.patch407 bytesgchauhan
#10 2979283-7.patch391 bytesakash.addweb
#6 2979283-6.patch1.44 KByadavarzoo
#5 Issues.png117.7 KBakash.addweb
#4 2979283-4.patch629 bytesyadavarzoo
#3 ReadMe_Error.png75.92 KBakash.addweb
#2 2979283.patch543 bytesyadavarzoo

Comments

yadavarzoo created an issue. See original summary.

yadavarzoo’s picture

yadavarzoo commented
Status: Needs work » Needs review
StatusFileSize
new 2979283.patch543 bytes

The theme has dependency on Bootstrap theme and commerce module. README is updated.

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Needs work
StatusFileSize
new ReadMe_Error.png75.92 KB

@yadavarzoo, Thank you for the patch but it doesn't work well for me as it doesn't follow the standards for the Readme template. You can refer this link: https://www.drupal.org/docs/develop/documenting-your-project/readme-temp... and also refer the attached screenshot.

yadavarzoo’s picture

yadavarzoo commented
Status: Needs work » Needs review
StatusFileSize
new 2979283-4.patch629 bytes

@Ronak.addweb, Thankyou for pointing it out. I have updated it.

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Needs work
StatusFileSize
new Issues.png117.7 KB

@yadavarzoo, Thank you for the patch again, but still, it doesn't match with Drupal standards. Kindly please refer this attached screenshot for found missing points.

yadavarzoo’s picture

yadavarzoo commented
Status: Needs work » Needs review
StatusFileSize
new 2979283-6.patch1.44 KB

README updated.

akash.addweb’s picture

akash.addweb commented

@yadavarzoo, Thanks for your efforts, I checked it using the ReadMe standards & found 1 small issue in the Configuration section. Need to wrap the text written in configuration by 82 characters.

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Needs work
yadavarzoo’s picture

yadavarzoo commented

@Ronak.addweb, Could you please add a patch for the issue that you are pointing? I am new to Drupal and I don't understand what is the issue in this readme file. And also see other files in the theme and contribute in making it better.

akash.addweb’s picture

akash.addweb commented
StatusFileSize
new 2979283-7.patch391 bytes

@arzooy97, As per the readme a template standard, here I attached the patch for the change of " Text manually word-wrapped within around 80 cols." You can refer to this patch. Let me know if any query.

Thank you.

akash.addweb’s picture

akash.addweb commented
Status: Needs work » Needs review
gchauhan’s picture

gchauhan commented
gchauhan’s picture

gchauhan commented
StatusFileSize
new 2979283-12.patch407 bytes
yadavarzoo’s picture

yadavarzoo commented
Priority: Major » Normal
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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