Create an endpoint with validation object for builder ERC721 #4

2021-06-03T05:28:07Z

olsau65 commented

2021-06-03 05:28:07 +00:00

(Migrated from github.com)

No description provided.

stasnepomn (Migrated from github.com) requested changes

2021-06-03 13:22:18 +00:00

src/controllers/tokens.ts

					
				@ -11,3 +11,3 @@

				class ValidTokenERC20 implements ERC20ContractData {

				class Erc20Validation implements ERC20 {

				  @IsString()

				  name: string

Неиспользуемые импорты лучше сразу удалять, чтобы не загромождать код

src/controllers/tokens.ts

					
				@ -34,10 +34,32 @@ class ValidTokenERC20 implements ERC20ContractData {

				  maxTokens: number

				}

Очень длинный путь для роутов, можно сократить до примитивных /erc20 и /erc721

Нужно стараться использовать наиболее короткие, но наиболее ёмкие названия для переменных

src/interfaces/erc721ContractData.ts

Достаточно назвать интерфейс ERC721, так будет короче и понятнее

Лучше использовать примитивы, встроенные в typescript, а не объекты из javascript (использовать string вместо String, number вместо Number etc.)