diff mbox

settings: Enable the REST API by default

Message ID 20170504185117.32749-1-stephen@that.guru
State Accepted
Headers show

Commit Message

Stephen Finucane May 4, 2017, 6:51 p.m. UTC
The release notes say this is already the case. As the API is now
functionally complete, there's no reason not to toggle this. Make it so.

Signed-off-by: Stephen Finucane <stephen@that.guru>
---
 patchwork/settings/base.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stephen Finucane May 4, 2017, 6:57 p.m. UTC | #1
On Thu, 2017-05-04 at 19:51 +0100, Stephen Finucane wrote:
> The release notes say this is already the case. As the API is now
> functionally complete, there's no reason not to toggle this. Make it
> so.
> 
> Signed-off-by: Stephen Finucane <stephen@that.guru>

I'm actually going to go right ahad and merge this so I can tag rc1. We
can revert after that if necessary.

Stephen
Daniel Axtens May 10, 2017, 7:14 p.m. UTC | #2
Hi Stephen,

Yep, I'm good with this - we really want to push the REST API so we can
drop xmlrpc sometime this decade :)

Regards,
Daniel 


> On Thu, 2017-05-04 at 19:51 +0100, Stephen Finucane wrote:
>> The release notes say this is already the case. As the API is now
>> functionally complete, there's no reason not to toggle this. Make it
>> so.
>> 
>> Signed-off-by: Stephen Finucane <stephen@that.guru>
>
> I'm actually going to go right ahad and merge this so I can tag rc1. We
> can revert after that if necessary.
>
> Stephen
>
> _______________________________________________
> Patchwork mailing list
> Patchwork@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/patchwork
diff mbox

Patch

diff --git a/patchwork/settings/base.py b/patchwork/settings/base.py
index e6d34ba..5f6aebe 100644
--- a/patchwork/settings/base.py
+++ b/patchwork/settings/base.py
@@ -224,7 +224,7 @@  NOTIFICATION_FROM_EMAIL = DEFAULT_FROM_EMAIL
 ENABLE_XMLRPC = False
 
 # Set to True to enable the Patchwork REST API
-ENABLE_REST_API = False
+ENABLE_REST_API = True
 
 REST_RESULTS_PER_PAGE = 30