| # |
Sep 13th 2019, 13:56 |
 ckjksl |
Would it be easier to save the file and read the lines you need rather than passing it through `type="hidden"`? |
| # |
Sep 13th 2019, 13:55 |
ckjksl |
@felipe.marinho Are you sure the 500,000 line CSV uploaded successfully? |
| # |
Sep 13th 2019, 13:27 |
info315 |
I will track it, looks like this is the best practice |
| # |
Sep 13th 2019, 13:15 |
dereuromark |
if you dont leverage it, you can also exclude for now :slightly_smiling_face: |
| # |
Sep 13th 2019, 13:11 |
info315 |
Ok |
| # |
Sep 13th 2019, 13:08 |
dereuromark |
usually people track it for project level. |
| # |
Sep 13th 2019, 13:00 |
info315 |
Should the file `Migrations/schema-dump-default.lock` be excluded from git or better get tracked? |
| # |
Sep 13th 2019, 12:59 |
felipe.marinho |
hey guys... I created an CSV uploader action, but I'm passing the CSV data to from 1 page to another using an input type="hidden". I tested with 30 lines, 100 lines, and it was OK BUT now the client is testing with a 500.000 lines CSV, and it's not working. What is another approach to pass this content and not screw up with the user's browser? |
| # |
Sep 13th 2019, 11:12 |
spriz |
Ah wait, it actually does in `PhpConfig` |
| # |
Sep 13th 2019, 11:11 |
spriz |
I guess the Configure::load does not fail even though there is no file? Or no? |
| # |
Sep 13th 2019, 10:57 |
lilhermit |
Thanks anyway guys |
| # |
Sep 13th 2019, 10:57 |
lilhermit |
maybe, I just like to keep my build/deployment as simple as possible |
| # |
Sep 13th 2019, 10:56 |
dereuromark |
Maybe. Writing out an empty file on deploy was always easier ^^ |
| # |
Sep 13th 2019, 10:51 |
lilhermit |
@dereuromark might be worth adding the loading of `app_local.php` in a file_exists on your blog? |
| # |
Sep 13th 2019, 10:50 |
lilhermit |
I thought I must have been missing something |
| # |
Sep 13th 2019, 10:50 |
spriz |
:tada: |
| # |
Sep 13th 2019, 10:50 |
lilhermit |
Yes just tested it and your correct ;) |
| # |
Sep 13th 2019, 10:49 |
lilhermit |
perfect then ignore everything I've said it won't break on deployment ;) |
| # |
Sep 13th 2019, 10:49 |
spriz |
unless you removed it, I guess it is :ok_hand: |
| # |
Sep 13th 2019, 10:49 |
spriz |
https://github.com/cakephp/app/blob/master/src/Console/Installer.php#L59 @lilhermit |
| # |
Sep 13th 2019, 10:49 |
lilhermit |
I did not know the `cp` is one on `composer install` ? |
| # |
Sep 13th 2019, 10:47 |
spriz |
actually when you do `composer install`, exactly that is done by default :slightly_smiling_face: |
| # |
Sep 13th 2019, 10:46 |
spriz |
Put all the sensitive stuff in env vars and move on :ok_hand: |
| # |
Sep 13th 2019, 10:46 |
spriz |
or just add `cp config/app.default.php config/app.php` to your deploy script :slightly_smiling_face: |
| # |
Sep 13th 2019, 10:42 |
dereuromark |
Easy enough to solve on project level though :) |
| # |
Sep 13th 2019, 10:41 |
dereuromark |
Bottom line: current approach with one big file is bad |
| # |
Sep 13th 2019, 10:41 |
dereuromark |
Fair enough |
| # |
Sep 13th 2019, 10:41 |
lilhermit |
Maybe I will do a PR |
| # |
Sep 13th 2019, 10:41 |
lilhermit |
@dereuromark The only problem I have with your blog post is add `Configure::load('app_local');` should be wrapped in a `file_exists` Personally I think `Configure::load` should support the second param being a config array so you could add a `only_if_exists` flag and then pass the merge and config key into the config array |
| # |
Sep 13th 2019, 10:34 |
dereuromark |
Someone could PR a more logical approach as I outlined. It also helps with core vs custom vs private data. |
| # |
Sep 13th 2019, 10:32 |
lilhermit |
@conehead Thanks yeah just wondered why it was default if it would instantly break on any checkout of the repo. Yeah I normally add `app.php` to repos just thought I'd review it on this new project |
| # |
Sep 13th 2019, 10:14 |
conehead |
But additionally: Noone is preventing you from actually adding the `app.php` to your repo if you really want to |
| # |
Sep 13th 2019, 10:13 |
conehead |
@lilhermit Maybe have a look at this: https://sandbox.dereuromark.de/pages/best-practices#config |
| # |
Sep 13th 2019, 09:54 |
lilhermit |
It feels like `app.default.php` should be included in the bootstrap.php |
| # |
Sep 13th 2019, 09:50 |
lilhermit |
I've never understood why `config/app.php` is gitignore'ed. obviously I understand why you don't want sensitive info in the repo but when you deploy to another environment the app auto fails because there is no app.php |
| # |
Sep 13th 2019, 09:28 |
spriz |
here maybe @info315 https://my.cakephp.org/join? |
| # |
Sep 13th 2019, 09:18 |
info315 |
Or better, how do i create the user for it? |
| # |
Sep 13th 2019, 09:16 |
info315 |
@admad Who can i access the admin area of https://github.com/cakephp/cakephp.org ? |
| # |
Sep 13th 2019, 08:54 |
conehead |
@info315 but thanks for your help. I am pretty sure I will need an API-Key shortly after |
| # |
Sep 13th 2019, 08:47 |
info315 |
@admad was a try to not spam this off topic stuff between real questions |
| # |
Sep 13th 2019, 08:43 |
admad |
@info315 if you use the silly threads i will just ignore your response :slightly_smiling_face: |