#48 - heroku deployment #70

Merged
Baakoma merged 34 commits from #48-heroku-deployment into main 2022-03-15 14:46:42 +01:00
Baakoma commented 2022-03-15 10:47:15 +01:00 (Migrated from github.com)

It should close #48

It should close #48
krzysztofrewak (Migrated from github.com) requested changes 2022-03-15 14:22:46 +01:00
@@ -7,6 +7,7 @@
"require": {
"php": "^8.1",
"ext-pdo": "*",
"ext-redis": "*",
krzysztofrewak (Migrated from github.com) commented 2022-03-15 14:21:36 +01:00

So do we need ext-redis or predis?

So do we need ext-redis or predis?
@@ -16,4 +12,3 @@
"calendar_id" => env("GOOGLE_CALENDAR_ID"),
"user_to_impersonate" => env("GOOGLE_CALENDAR_IMPERSONATE"),
];
krzysztofrewak (Migrated from github.com) commented 2022-03-15 14:21:57 +01:00

I would pu it into storage directory.

I would pu it into storage directory.
krzysztofrewak (Migrated from github.com) commented 2022-03-15 14:22:30 +01:00

Could you provide here some default values if they're not secret ones?

Could you provide here some default values if they're not secret ones?
Baakoma (Migrated from github.com) reviewed 2022-03-15 14:26:43 +01:00
@@ -16,4 +12,3 @@
"calendar_id" => env("GOOGLE_CALENDAR_ID"),
"user_to_impersonate" => env("GOOGLE_CALENDAR_IMPERSONATE"),
];
Baakoma (Migrated from github.com) commented 2022-03-15 14:26:43 +01:00

Heroku buildpack creates this file in root directory and we cannot change this location.

Heroku buildpack creates this file in root directory and we cannot change this location.
Baakoma (Migrated from github.com) reviewed 2022-03-15 14:28:47 +01:00
@@ -7,6 +7,7 @@
"require": {
"php": "^8.1",
"ext-pdo": "*",
"ext-redis": "*",
Baakoma (Migrated from github.com) commented 2022-03-15 14:28:47 +01:00

ext-redis

ext-redis
krzysztofrewak (Migrated from github.com) reviewed 2022-03-15 14:29:44 +01:00
@@ -16,4 +12,3 @@
"calendar_id" => env("GOOGLE_CALENDAR_ID"),
"user_to_impersonate" => env("GOOGLE_CALENDAR_IMPERSONATE"),
];
krzysztofrewak (Migrated from github.com) commented 2022-03-15 14:29:43 +01:00

Okok.

Okok.
krzysztofrewak (Migrated from github.com) approved these changes 2022-03-15 14:44:10 +01:00
Sign in to join this conversation.