Skip to content

It should be possible to set service_mode to something else than 1#12

Merged
b4k3r merged 1 commit intosocketry:masterfrom
gleseur:master
Mar 25, 2015
Merged

It should be possible to set service_mode to something else than 1#12
b4k3r merged 1 commit intosocketry:masterfrom
gleseur:master

Conversation

@gleseur
Copy link
Copy Markdown

@gleseur gleseur commented Feb 10, 2015

Hi,

We don't want to set service_mode to '1' for all our records so it's nicer to have it as a default value

Thanks,

@gleseur gleseur changed the title It should be possible to set service_mode tp something else than 1 It should be possible to set service_mode to something else than 1 Feb 10, 2015
@pencoyd
Copy link
Copy Markdown

pencoyd commented Feb 10, 2015

FYI, you might want to review the new APIs here https://www.cloudflare.com/docs/next/

We'll continue to support the current Client API for many months to come, of course, but you might find that useful. We haven't "promoted" the new APIs yet, but they are active and in use.

@alex-fedorov
Copy link
Copy Markdown

Any news on this one? I have run in this problem, where it is not possible to create record with service_mode => 0. Though it is possible to update it to this state, which means, that to create record with service_mode => 0, I need to do 2 things:

  • create record
  • update record with service_mode => 0

Which is frustrating.

@pencoyd
Copy link
Copy Markdown

pencoyd commented Mar 25, 2015

I'd encourage use of the APIs at https://api.cloudflare.com instead. We're not planning to make further changes to the v1 API, as the newer APIs (v4) address this use case.

b4k3r added a commit that referenced this pull request Mar 25, 2015
It should be possible to set service_mode to something else than 1
@b4k3r b4k3r merged commit 91e7182 into socketry:master Mar 25, 2015
@ryush00 ryush00 mentioned this pull request May 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants