а теперь увеличь вложенность, добавь хуки before/after в каждом скоупе и пожалуйста в rest-стайл

про хуки да, согласен, что они удобные, но про остальное - зачем все городить в один файл в принципе? Если роутов мало, мало вложенности, то ок, но когда всего много, то надо бы их разбивать

и вот тут я не вижу чем твоя либа кардинально отличается от экспресс роутера, окромя хуков

Ахахах, нет, ребята, вы не шарите

Ахахах, нет, ребята, вы не шарите

сказал Келин )0))

@Fl0pZz ты начитался опять статеек не понял, о чем в них пишут, и полез повторять Ты не понимаешь, где стоит усложнять и ограничивать, а где можно сделать просто

@Fl0pZz ты начитался опять статеек не понял, о чем в них пишут, и полез повторять Ты не понимаешь, где стоит усложнять и ограничивать, а где можно сделать просто

вот у меня вообще ни разу не сложно

Я понимаю, что если сделать простым ВСЕ, то будет расхлябанность и все будет через жопу Но это не повод писать какую-то херь

и я не просто так сходу взял и написал, я сначала у 3х либ исходники изучил

Впрочем, это ваше дело

и вот тут я не вижу чем твоя либа кардинально отличается от экспресс роутера, окромя хуков

я ещё не впилил чистый resources для множества ресурсов и когда их можно композировать вкладывать друг в друга как угодно типа есть albums/:albumId есть albums/:albumId/photos/:photoId/comments/:commentId и чтобы хуками делать проверки и вытаскиывать нужные объекты

и я не просто так сходу взял и написал, я сначала у 3х либ исходники изучил

Я тебе об этом выше написал

/stat@combot

combot.org/chat/-1001041204341

и потом минимум 4 больших измнения в АПИ внес

@Fl0pZz ты начитался опять статеек не понял, о чем в них пишут, и полез повторять Ты не понимаешь, где стоит усложнять и ограничивать, а где можно сделать просто

ну ясно...

и потом минимум 4 больших измнения в АПИ внес

причем не каждый день вносил, а спустя какое-то время, чтобы в голове все улеглось

я ещё не впилил чистый resources для множества ресурсов и когда их можно композировать вкладывать друг в друга как угодно типа есть albums/:albumId есть albums/:albumId/photos/:photoId/comments/:commentId и чтобы хуками делать проверки и вытаскиывать нужные объекты

а, ну тогда возможно, надо будет посмотреть позже :) потому что пока профита особо не видно

а, ну тогда возможно, надо будет посмотреть позже :) потому что пока профита особо не видно

ну она не релизнулась же ещё, даже поддержки koa нет ))

Проблемы с наследованием только ухудшатся, потому что больше вариантов. Но неиспользовать сейчас typescript или flow — это добровольный поиск проблем

Я как бы, как раз и хочу избавиться от многовариантности того же наследования, которое присутствует в "чистом" JS. Развев TS не решает этой проблемы ?

а, ну тогда возможно, надо будет посмотреть позже :) потому что пока профита особо не видно

я немного изменил ридми https://createrest.js.org/

переименуй вот эти e => во что-то более понятное или это сделаю я :D

я немного изменил ридми https://createrest.js.org/

кстаи, а почему так? можно осовременить function before1() { console.log('before1()') }

переименуй вот эти e => во что-то более понятное или это сделаю я :D

например во что?

кстаи, а почему так? можно осовременить function before1() { console.log('before1()') }

можно)

например во что?

запилю ПР, как времечко будет

:)

можно)

ктстаи, еще сделай более явные глобальные хуки

ктстаи, еще сделай более явные глобальные хуки

конкретнее??

вот это прям не выразительно

мне пришлось напрячься, чтобы понять, что это глобальные хуки

лучше r.beforeEach( ?

лучше r.beforeEach( ?

да

мне пришлось напрячься, чтобы понять, что это глобальные хуки

просто они внутри любого scope есть

before лучше, чем beforeEach

вот он минус деланья их через функции

вот он минус деланья их через функции

у себя я стырил composer из koa и это позволио мне избавитьсч от before и after оставив только hook

т.е у меня хуки выглядят как middleware в коа

мне пришлось напрячься, чтобы понять, что это глобальные хуки

https://github.com/atomixinteractions/createrest/issues/5 @vgoloviznin @Fl0pZz

https://github.com/atomixinteractions/createrest/issues/5 @vgoloviznin @Fl0pZz

вообще хотелось бы именно глобальные переименовать

да даже просто сделай алисаы на before и after, думаю этого будет достаточно

вообще хотелось бы именно глобальные переименовать

да тут нет глобальных но я подумаю как можно сделать понятнее

а, ты это указал у себя

короче, надо запилить живой boilerplate

хм, погодика, я кажется понял в чем главная трабла с твоей декларацией: e.get('/', get1) e.get('/foo', get2) по сути это должно выглядить вот так: / foo

но это не видно

хм, погодика, я кажется понял в чем главная трабла с твоей декларацией: e.get('/', get1) e.get('/foo', get2) по сути это должно выглядить вот так: / foo

это выглядит так GET / -> get1 GET /foo -> get2

да, но когда ты обращешься к /foo я напимер ожидаю, что должны отрабоать "хуки" для корешка и хуки для фоо

да, но когда ты обращешься к /foo я напимер ожидаю, что должны отрабоать "хуки" для корешка и хуки для фоо

да, так и будет

но из декларации это не видно

суть в том, что хуки добавляются на текущий уровень и во все вложенные уровне

во ВСЕ

надо это текстом описать как-то...

а у тебя meta инфы нету?

а у тебя meta инфы нету?

meta?

да

это какой?

это какой?

http://router.vuejs.org/en/advanced/meta.html

эм конечно нет

здесь нет понятии auth например)

здесь нет понятии auth например)

почему? это же было бы удобно

сразу в хуке проверил, авторизирован пользователь или нет, сразу отредежектил если нет

сразу в хуке проверил, авторизирован пользователь или нет, сразу отредежектил если нет

ща покажу

requiredAccess можно перенести в метод Profile.beforeEach()

requiredAccess можно перенести в метод Profile.beforeEach()

зачем ты для get и post добавил requiredAccess? они же у тебя от beforeEach и так добавятся

ERROR: S client not available

зачем ты для get и post добавил requiredAccess? они же у тебя от beforeEach и так добавятся

разные права нужны))

а, все, увидил

А почему в хуках у тебя before, а в контроллере beforeEach?

@sergeysova дай ссыль на свою репу с rest express route

А почему в хуках у тебя before, а в контроллере beforeEach?

да, мысль хорошая, наверное надо изменить

@sergeysova дай ссыль на свою репу с rest express route

createrest.js.org atmx.in/rest

подвалил апдейт ридми, который хотел :)

подвалил апдейт ридми, который хотел :)

о, гуд! отлично!

в тестах бы тоже поменять имена, потому что вообще ничего не понятно :)

да, пожалуй сменю

Например тут ничего не понятно

http://prntscr.com/ffhtpw

http://prntscr.com/ffhtpw

ты тот человек, который смог выразить мои мысли про тесты -_-

спасибо, ребят

Я вообще стараюсь придерживаться assign-act-assert подхода, тогда понятно что на вход дается и что должно получиться

ну и названия тестов тоже стараюсь, чтобы отражали то, что должно быть

ну и названия тестов тоже стараюсь, чтобы отражали то, что должно быть

можешь пример показать?

это трудно, я так и не смог своим тестам придумать адекватные названия

это трудно, я так и не смог своим тестам придумать адекватные названия

+++

если на исходники посмотреть, то у меня названия тестов это названия тестируемой функции

можешь пример показать?

типа такого http://prntscr.com/ffhwjz

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

ну и иногда дополнительные описания добавляю, типо base или base + params and data

структура примерно такая

http://prntscr.com/ffhx48

где applyStickers - название метода

обновил тесты

постараюсь на них тож ПР залить

:)

https://github.com/atomixinteractions/createrest/blob/master/test/index.test.js