Closed (fixed)
Project:
Financial Calculators
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
23 Jul 2019 at 13:06 UTC
Updated:
15 Aug 2019 at 14:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shubham.prakash commentedThis patch should fix the issue.
Comment #3
anmolgoyal74 commentedCS issue: If the line declaring an array spans longer than 80 characters, each element should be broken into its own line.
Comment #4
shubham.prakash commentedThanks for the review, this patch should fix the issue.
Comment #5
alonaoneill commentedReviewed the patch for spelling and grammar. Everything looks good, but:
Text should manually word-wrap within around 80 cols.
Also "Configuration" part is missing. The configuration section (required) is necessary even when little configuration is required.
I would add something like:
1. Navigate to Administration > Extend and enable the module.
2. Navigate to Administration > Structure > Block layout and "Place block" from "Financial Calculators" category.
Comment #6
shubham.prakash commentedHere is the patch for same.
Comment #7
anmolgoyal74 commentedIt's better now. Thanks for the patch.
Comment #9
gaurav.kapoor commented