feat/#8-write-data-type-between-ledgers

[danielvo11]

• remove uint64_t type
• add int, bool, double and address type

Data type	EOS	ICON
boolean	bool	bool
number	double	int
uint64	uint64_t	int
uint32	uint32_t	int
uint16	uint32_t	int
string	string	string
account	name	Address

[quocle108]

Should change address to account in abstract class for easy understand

[manh-vv]

It looks great thank @danielalvess. Since this MR is going to make a huge conflict with #44, I prefer to review and merge #44 first.

[quocle108]

Pls resolve conflic

[manh-vv]

@danielalvess

I don't think we have to introduce new names for the data type. #53 (comment). Since data type in EOS can be considered as a superset of data type in ICON, we can use names of data types in EOS in our yaml file. For example:

Data type	EOS	ICON
bool	bool	bool
string	string	string
number	uint64_t	int
uint64_t	uint64_t	int
uint32_t	uint32_t	int
uint16_t	uint16_t	int
account	name	Address

It is ok with number but please keep the default of number in EOS to be uint_64.

[manh-vv]

The other things look good to me.

[manh-vv]

Agreed that the default value of number is double.