nodosjs / nodos Goto Github PK

Nodos should have cli command helping the user search through code for comments with a specific keyword like here: https://guides.rubyonrails.org/command_line.html#bin-rails-notes

Task: root (из рельсового роутинга)

Push workflow references a Make rule lint-ci which is not implemented in Makefile

For more information check #133

Можно добавить такое правило линтера

    "rules": {
        "no-unused-vars": [2, {"args": "after-used", "argsIgnorePattern": "^_"}]
    }

https://eslint.org/docs/2.0.0/rules/no-unused-vars

И неиспользуемые переменные начинать с _

Кажется это уже сделано?

Example command npx nodos generate model user/comment

nodos new generate an "undefined" directory.

Nodos should have cli command which shows the user his application environment like here: https://guides.rubyonrails.org/command_line.html#bin-rails-about

After generating a new project it is not obvious that the developer should run npm install to start work with the project (use nodos commands, for example)

Nodos needs a logo and simple identity like color palette for the documentation site

Кейс: приложение находится за реверс прокси. Само приложение работает на любом порту

Открываем приложение например на 80/443 порту. Отправляем форму (например создание пользователя), нас перенаправляет на localhost:8080, получаем ошибку CSRF Mismatch, потому что нас будет перенаправлять на
localhost:<port>

Как должно быть

У пользователя формируются ссылки с хостом с которого пришел или базовый урл приложения задается в переменных окружения.

Nodos should give users the ability to use cli associated with the database like here: https://guides.rubyonrails.org/command_line.html#bin-rails-dbconsole

Nodos should have a simple command to delete or destroy generated resources like here: https://guides.rubyonrails.org/command_line.html#bin-rails-destroy

Slack chat archive
kirill.mokevnin:
я думаю что глубокий мерж надо делать на уровне addOption
то есть мы туда тупо добавляем дефолт
а оно дальше мержит
то есть я бы назвал app.setDefaultConfig(key, value)
вот так
ну и оно перезаписало только если что-то не так
в смысле только для отсутствующих
и получилось бы так

app.setDefaultConfig(defaultConfig);
app.config....

1. https://nodosjs.github.io/ - docusaurus cards need to be removed
2. docusaurus logo needs to be removed from all pages

Как мигрировать (работать с бд?)
После добавления в routes надо перезапускать сервер
В ридми нет примера как генерировать модельки/контроллер
Нет примера работы с шаблоном/работающего круда
Сгенерированный шаблон не наследуется от extends /layouts/application.pug
в app/controllers/root.js второй параметр называется response, в app/controllers/users.js - reply
в сгенерированном контроллере - response
Непонятно назначение директории app/javascript

Example

buildSession        GET    /session/build                          browser
sessions            POST   /session                                browser
session             GET    /session                                browser
editSession         GET    /session/edit                           browser
session             PATCH  /session                                browser
session             PUT    /session                                browser
session             DELETE /session                                browser

What we have:

session               DELETE /session                                browser

What should we have:

destroySession        DELETE /session                                browser

We need to know where the middleware comes from - local, db, view or from other extension

What we have

browser:
  - checkMigration
  - protectFromForgery
  - setCurrentUser

What we want (just suggestion, can be implemented differently)

  - db/checkMigration
  - view/protectFromForgery
  - local/setCurrentUser

Первый PR смержен #5, теперь надо докручивать генератор:

1. Чтобы nodos new принимал аргумент и генерил приложение с таким именем или по такому пути: nodos new myawesomeapp/nodos new ~/users/ololoev/myawesomeapp
2. Чтобы генерился весь набор файлов (cм. example), а не только один файл как сейчас.