-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added notes table #52
Conversation
case "INSTRUMENT_MIN": | ||
return "Reflects the minimum amount of an adjustable warrant."; | ||
default: | ||
return "There is no specified source for this warrant"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about "There is no specified source for the amount of this warrant"?
case "MACHINE_ESTIMATED": | ||
return "Reflects the machine estimated amount of an adjustable warrant."; | ||
case "INSTRUMENT_FIXED": | ||
return "Reflects the fixed amount of an adjustable warrant."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's fixed it's not adjustable.
@@ -35,6 +35,7 @@ class Model implements WorkbookModel { | |||
private ratioCalculator = new ConversionRatioCalculator(); | |||
public warrantStockIds: Set<string> = new Set(); | |||
public nonPlanStockIds: Set<string> = new Set(); | |||
public warrantsSources = new Map<string, string>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Data structure that contains the warrant stock ids and the corresponding quantity_source
@@ -65,6 +65,17 @@ class ExcelJSLinePrinter { | |||
public setColWidth(col: number, width: number) { | |||
this.worksheet.getColumn(col).width = width; | |||
} | |||
|
|||
public mergeCells( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method to merge cells. Useful for the notes
header
SHOW
Fixes issue #50