Log message #4203086

# At Username Text
# 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: