| # |
Aug 23rd 2019, 16:52 |
mogpusse |
Hello |
| # |
Aug 23rd 2019, 16:16 |
 admad |
@davorminchorov unless you really want the mail to travel through the interwebs, you can use something like mailhog on local network |
| # |
Aug 23rd 2019, 16:14 |
admad |
We should fix that in 3.next |
| # |
Aug 23rd 2019, 16:06 |
dereuromark |
I would say, yes |
| # |
Aug 23rd 2019, 14:43 |
rchavik |
it should by default escape the title, isn't it? |
| # |
Aug 23rd 2019, 14:36 |
dereuromark |
might be historically |
| # |
Aug 23rd 2019, 14:35 |
rchavik |
BreadcrumbsHelper::add() does not auto escape? |
| # |
Aug 23rd 2019, 14:05 |
davorminchorov |
I am mainly setting this up for mailtrap for local testing because the rate limit is 3 emails per second, I think it would be fine for other SMTP providers like gmail etc. |
| # |
Aug 23rd 2019, 14:03 |
ricksaccous |
i dunno, but yeah it's possible |
| # |
Aug 23rd 2019, 14:03 |
ricksaccous |
or you could also just add taht metadata in and check against it to say worker hold on a bit |
| # |
Aug 23rd 2019, 14:02 |
ricksaccous |
oh cool then yeah you should be able to mess with the workers and worker timings to set the rates |
| # |
Aug 23rd 2019, 14:02 |
dereuromark |
if it fails it would just resend. But yeah, you could build in a bit of rate limiting. |
| # |
Aug 23rd 2019, 14:02 |
davorminchorov |
Yeah, I am currently using cakephp-queue by dereuromark, I'll see the config if it does have any rate limits |
| # |
Aug 23rd 2019, 14:01 |
ricksaccous |
by default it just attempts to send all emails at once i believe |
| # |
Aug 23rd 2019, 14:01 |
ricksaccous |
you'd have to set things up so the plugin handles email sending though |
| # |
Aug 23rd 2019, 14:00 |
ricksaccous |
using the plugin you can specify a rate |
| # |
Aug 23rd 2019, 14:00 |
ricksaccous |
@davorminchorov if you need something like that I'd recommend the Queue plugin |
| # |
Aug 23rd 2019, 13:59 |
davorminchorov |
Is there a rate limit configuration for emails in CakePHP? |
| # |
Aug 23rd 2019, 13:54 |
dereuromark |
I would also think that the route itself is enough in that case |
| # |
Aug 23rd 2019, 13:52 |
neon1024 |
Only advantage I can see is that the controller method tells me which methods it likes right in the code |
| # |
Aug 23rd 2019, 13:51 |
neon1024 |
i don’t have fallback routes in this prefix, so I’m guessing checking in the controller is redundant as the route just won’t match right? |
| # |
Aug 23rd 2019, 13:51 |
neon1024 |
Feels like I’m doing the same thing from different ends |
| # |
Aug 23rd 2019, 13:50 |
neon1024 |
If I specify an http method in my route as `->setMethods(['get'])` for example, do I need to check `$this->getRequest->allowMethod('get')` in my controller? |
| # |
Aug 23rd 2019, 13:34 |
admad |
shell classes are deprecated, replaced by command classes. It is planned to make commands lib standalone but markstory hasnt got around to it |
| # |
Aug 23rd 2019, 13:32 |
val |
@admad Hi, would it be possible to have a standalone 3.x shell classes library like standalone ORM https://github.com/cakephp/orm? |
| # |
Aug 23rd 2019, 12:49 |
admad |
if you absolutely want to support both uuid and int then make 2 fk fields and configure which field to use in association |
| # |
Aug 23rd 2019, 12:48 |
slackebot2 |
'length' => 64, ``` |
| # |
Aug 23rd 2019, 12:48 |
slackebot2 |
->addColumn('parent_asset_id', 'integer', [ 'null' => true, 'default' => null, ]) - ->addColumn('foreign_key', 'string', [ - 'null' => true, 'default' => null, 'length' => 36, + ->addColumn('foreign_key', 'integer', [ + 'null' => true, 'default' => null, ]) ->addColumn('model', 'string', [ 'null' => true, 'default' => null, |
| # |
Aug 23rd 2019, 12:48 |
rchavik |
solved: ``` diff --git a/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php b/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php index 2be8a93f..b1a24282 100644 --- a/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php +++ b/FileManager/config/Migrations/20131130074112_AssetsInitialMigration.php @@ -64,8 +64,8 @@ class AssetsInitialMigration extends AbstractMigration { |
| # |
Aug 23rd 2019, 12:47 |
admad |
hehe, it happens, something we get too focused trying to solve things one way |
| # |
Aug 23rd 2019, 12:47 |
rchavik |
:face_palm: |
| # |
Aug 23rd 2019, 12:47 |
rchavik |
it never occured to me to change the damn schema |
| # |
Aug 23rd 2019, 12:47 |
rchavik |
you know.. |
| # |
Aug 23rd 2019, 12:46 |
admad |
ah, noble cause :slightly_smiling_face: |
| # |
Aug 23rd 2019, 12:45 |
rchavik |
yeah, this was trying to support integer and char(36) for uuids |
| # |
Aug 23rd 2019, 12:45 |
admad |
not sure how db db will handle if the value isn't quoted |
| # |
Aug 23rd 2019, 12:44 |
admad |
overriding field type might creating problems when saving data |
| # |
Aug 23rd 2019, 12:44 |
admad |
but best is to avoid creating unnecessary problems which you have to solve :slightly_smiling_face: |
| # |
Aug 23rd 2019, 12:43 |
rchavik |
noted, i will try to change the field |
| # |
Aug 23rd 2019, 12:43 |
admad |
you can try overriding the field type in table table schema in Table::_initalizeSchema() |
| # |
Aug 23rd 2019, 12:41 |
admad |
*same as |