| # |
Apr 21st 2017, 13:41 |
Neon1024 |
10/10 would version again |
| # |
Apr 21st 2017, 13:41 |
Neon1024 |
I did, it’s worked great for me :) |
| # |
Apr 21st 2017, 13:41 |
 birdy247 |
So you used prefix routes? |
| # |
Apr 21st 2017, 13:41 |
Neon1024 |
So I can have \App\Controller\Api\V3\ProductsController |
| # |
Apr 21st 2017, 13:41 |
Neon1024 |
birdy247, The trick I used was to use /api as a prefix, then have version prefixes under that |
| # |
Apr 21st 2017, 13:41 |
birdy247 |
so in cake, how would I handle versions? |
| # |
Apr 21st 2017, 13:40 |
technofection |
@hmic I am still stuck :( |
| # |
Apr 21st 2017, 13:39 |
birdy247 |
Neon1024 ah yes, thats what I am used to |
| # |
Apr 21st 2017, 13:39 |
hmic |
admad: actually i'm not. *but* it is the default config in your readme and for me, with just the app template and copyandpaste it does not require auth on 3.4.5 |
| # |
Apr 21st 2017, 13:39 |
Neon1024 |
Usually something like example.com/api/:version/:endpoint |
| # |
Apr 21st 2017, 13:39 |
Neon1024 |
Been to http://jsonapi.org ? |
| # |
Apr 21st 2017, 13:38 |
birdy247 |
like domain.com/api/view/3.json |
| # |
Apr 21st 2017, 13:38 |
birdy247 |
I was thinking of a specific scope |
| # |
Apr 21st 2017, 13:38 |
Neon1024 |
Can I have Chronos output an array? |
| # |
Apr 21st 2017, 13:37 |
Neon1024 |
Very strange |
| # |
Apr 21st 2017, 13:37 |
Neon1024 |
Hm, I just can’t get it to pass the if() |
| # |
Apr 21st 2017, 13:37 |
birdy247 |
cake makes this super easy :slightly_smiling_face: |
| # |
Apr 21st 2017, 13:37 |
birdy247 |
I am wanting to create an API for my app |
| # |
Apr 21st 2017, 13:37 |
birdy247 |
Hi |
| # |
Apr 21st 2017, 13:35 |
juniorionut |
don't worry, will do, thanks! |
| # |
Apr 21st 2017, 13:34 |
admad |
All i can tell you is read GH docs, i don't have time right now to walk you through the process |
| # |
Apr 21st 2017, 13:34 |
juniorionut |
i usually make new branches for changes |
| # |
Apr 21st 2017, 13:34 |
juniorionut |
so far no, relatively new to GH, read some doc , tried to use a fork but it game the message that everytrhing is merged |
| # |
Apr 21st 2017, 13:34 |
admad |
i suggest you read GH docs on how to create a PR. |
| # |
Apr 21st 2017, 13:33 |
admad |
two different branch names are not necessary, the master of your fork can be compared with master of main repo. Although its' good practice to always make a new branch for changes |
| # |
Apr 21st 2017, 13:32 |
admad |
have you opened pull requests on GH before? :slightly_smiling_face: |
| # |
Apr 21st 2017, 13:32 |
juniorionut |
You’ll need to use two different branch names to get a valid comparison. |
| # |
Apr 21st 2017, 13:31 |
admad |
huh? |
| # |
Apr 21st 2017, 13:31 |
juniorionut |
@admad, i cannot create a new PR, source and comparison branch are only master |
| # |
Apr 21st 2017, 13:30 |
technofection |
For example, Persons hasMany PhoneNumbers, when if I fetch a person entity by doing Persons->get(), then how do I exactly, inject a new number in the PhoneNumbers associated model? |
| # |
Apr 21st 2017, 13:29 |
technofection |
Can anyone tell how to patch an entity and insert a new record in hasMany associated model ? |
| # |
Apr 21st 2017, 13:28 |
admad |
though it should work with `checkAuthIn => Controller.initialize` too |
| # |
Apr 21st 2017, 13:26 |
admad |
you need to use that config only if trying to check auth info in beforeFilter(), which i doubt you are |
| # |
Apr 21st 2017, 13:25 |
hmic |
admad: i'm just struggling with jwt, if using checkAuthIn => Controller.initialize like in your git readme, it does not work, so allows everything. without checkAuthIn it works like i would have expected. am i wrong? cake-3.4.5 btw. |
| # |
Apr 21st 2017, 13:24 |
juniorionut |
make sense :slightly_smiling_face: |
| # |
Apr 21st 2017, 13:23 |
Neon1024 |
Might refactor that |
| # |
Apr 21st 2017, 13:23 |
Neon1024 |
I should probably type hint against an interface really |
| # |
Apr 21st 2017, 13:23 |
Neon1024 |
I take it all back |
| # |
Apr 21st 2017, 13:21 |
admad |
if you don't want some other sorry soul to suffer too then yes :slightly_smiling_face: |
| # |
Apr 21st 2017, 13:21 |
Neon1024 |
dereuromark, It is true! TIL https://3v4l.org/lpJ4X |
| # |
Apr 21st 2017, 13:20 |
juniorionut |
@admad so do you want me to creata a new pull request? |