Merge pull request #75 from RenovoSolutions/attachment_message_log

MessageLog: use strong types instead of strings

Author: Josh Graber

source/components/messageLog/editableMessageLog.ts

@@ -55,7 +55,7 @@ export class EditableMessageLogController {
 		this.savingMessage = true;
 		var message: string = this.newMessage;
 		this.newMessage = '';
-		return this.messageLogService.addMessage(message);
+		return this.messageLogService.addMessage({ message: message });
 	}
 }
 

source/components/messageLog/messageLog.service.ts

@@ -9,10 +9,10 @@ export var factoryName: string = 'messageLog';
 export var defaultPageSize: number = 10;
 
 export interface IMessage {
+	id?: number;
 	message: string;
-	createdBy: string,
-	createdDate: string,
-	createdTime: string,
+	createdBy?: string,
+	createdDate?: Date,
 }
 
 export interface IGetMessagesResult {
@@ -21,12 +21,12 @@ export interface IGetMessagesResult {
 }
 
 export interface IMessageLogDataService {
-	saveMessage(message: string): ng.IPromise<void>;
+	saveMessage(message: IMessage): ng.IPromise<void>;
 	getMessages(startFrom: number, quantity: number): ng.IPromise<IGetMessagesResult>;
 }
 
 export interface IMessageLog {
-	addMessage(message: string): ng.IPromise<void>;
+	addMessage(message: IMessage): ng.IPromise<void>;
 	visibleMessages: IMessage[];
 
 	getNextPage(): ng.IPromise<void>;
@@ -85,7 +85,7 @@ export class MessageLog {
 	}
 	/* tslint:enable */
 
-	addMessage(message: string): ng.IPromise<void> {
+	addMessage(message: IMessage): ng.IPromise<void> {
 		return this.dataService.saveMessage(message).then((): void => {
 			this.getTopPage();
 		});