Add Countries API #2
Conversation
AlexeyTeterin
force-pushed
the
task/countries-API
branch
from
3241832
to
524bbc7
Compare
src/models/sights.ts
Outdated
| countryID: { type: String, required: true }, | ||
| titleEN: { type: String, required: true, unique: true }, | ||
| titleRU: { type: String, required: true, unique: true }, | ||
| titleDE: { type: String, required: true }, |
There was a problem hiding this comment.
titleDE: { type: String, required: true }, тоже unique: true. Но, наверное, лучше id уникальным оставить. М б ситуация когда на разных языках слово пишется одинаково
src/routes/countries.ts
Outdated
|
|
||
| export const countriesRouter: express.Router = express.Router() | ||
|
|
||
| countriesRouter.options( |
There was a problem hiding this comment.
Cors проверки у нас на верхнем уровне подключены, теоретически действуют на все запросы. Попробуй убрать запрос options.
| infoRU: string | ||
| infoDE: string | ||
| pictureURL: string | ||
| } |
There was a problem hiding this comment.
не хватает пустой строки
| currency: string | ||
| pictureURL: string | ||
| videoURL: string | ||
| sights: ISight[] |
AlexeyTeterin
force-pushed
the
task/countries-API
branch
from
0195dc4
to
2ad1e04
Compare
| currency: string | ||
| pictureURL: string | ||
| videoURL: string | ||
| sights: ISight[] |
There was a problem hiding this comment.
Запятых в конце строки отсыпь, пожалуйста )
| infoEN: string | ||
| infoRU: string | ||
| infoDE: string | ||
| currency: string |
There was a problem hiding this comment.
Есть подозрение, что currency - это сложный тип с идентификатором валют и наименованиями на разных языках, может быть, символом. Предлагаю это обсудить
There was a problem hiding this comment.
Предлагаю делать тут трехбуквенное обозначение - USD, RUB...
src/routes/countries.ts
Outdated
|
|
||
| export const countriesRouter: express.Router = express.Router() | ||
|
|
||
| countriesRouter.options( |
There was a problem hiding this comment.
options точно надо вручную обрабатывать?
No description provided.