JohnCMS | Разработка следующей версии

Тема закрыта
38.61K
.
¯\_(ツ)_/¯
AlkatraZ, Хорошенькая)
.
Бермудский
Треугольник
Спасибо девушке вьетнамке.
Прикрепленные файлы:
.
КоханийВолодимир
Хотел кое-что запушить, но увы,
ERROR: Permission to simba77/johncms.git denied to krchVova.
Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
.
╭∩╮ (`-`) ╭∩╮
# КоханийВолодимир (19.11.2019 / 19:08)
Хотел кое-что запушить, но увы,
Уважаемый, а ты неправильно делаешь.
Ты же должен сделать следующее:
1) Форкнуть репозиторий к себе. Там уже у тебя будут нужные права.
2) Внести желаемые изменения (доработки).
3) У тебя там появится кнопка "new pull request"
Ну и шлешь в основной реп то, что ты сделал.
.
КоханийВолодимир
Сорри, это у меня первый раз
.
╭∩╮ (`-`) ╭∩╮
# КоханийВолодимир (19.11.2019 / 19:21)
Сорри, это у меня первый раз
Не страшно, у любого когда-то бывает в первый раз.
Всегда обращайся за советом, если понадобится.
.
#НетВойне #NoWar #НіВійні
PaRtiZzaN, +поддержу насчет девушки)
.
╭∩╮ (`-`) ╭∩╮
Добавлено: 19.11.2019 / 22:40
КоханийВолодимир, уважаемый, там твой PR не приняли, но сама идея с хелпером app() что ты предложил полезная, но при условии, если хелпер доработать. Там в коментарии к твоему PR я объяснил что нужно.
Добавляй только этот хелпер и все.
Модули не трогай и лишние хелперы не добавляй, они не нужны, у нас для этого есть константы.

Добавлено: 19.11.2019 / 22:41
Только обзови хелпер не app() а di() это в полном названии Dependency Injection

Добавлено: 19.11.2019 / 22:59
Все, уже не надо, сделал сам, там всего 3 строки.
закомитил от твоего имени, ты ведь предложил.
.
JohnCMS
Дебуска из бетнама сделай пожалуйста опобещение везду
.
Бывший человек
товарисчи а раз вы разметку переписываете - в форме ввода предвидятся нововведения?
Всего: 1376
Кураторы: AlkatraZ