Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updtae createTodoByTodolist? #19

Open
cms09 opened this issue Jul 25, 2014 · 3 comments
Open

Updtae createTodoByTodolist? #19

cms09 opened this issue Jul 25, 2014 · 3 comments

Comments

@cms09
Copy link

cms09 commented Jul 25, 2014

Hello Markri,

in the basecamp api you can add a date and assign a person to a todo list.
Maybe you can update your service with that feature?

'createTodoByTodolist' => array(
            'httpMethod' => 'POST',
            'uri' => 'projects/{projectId}/todolists/{todolistId}/todos.json',
            'parameters' => array(
                'projectId' => array(
                    'location' => 'uri',
                    'description' => 'Project id',
                    'required' => true
                ),
                'todolistId' => array(
                    'location' => 'uri',
                    'description' => 'Todo list id',
                    'required' => true
                ),
                "content" => array(
                    "location" => "json",
                    "type" => "string",
                    'required' => true,
                ),               
                "due_at" => array(
                    "location" => "json",
                    "type" => "string",
                    'required' => false,
                ),
                "assignee" => array(
                    "location" => "json",
                    "type" => "array",
                    'required' => false,
                )
            )
        ),
@cms09
Copy link
Author

cms09 commented Oct 13, 2014

Wonderfull, thank you.

@cms09 cms09 closed this as completed Oct 13, 2014
@cms09
Copy link
Author

cms09 commented Oct 14, 2014

Hi Markri,

Sorry that i have to reopen it, but you forget the due date field.

"due_at" => array(
"location" => "json",
"type" => "string",
'required' => false,
),

Thank you,
Chris

@cms09 cms09 reopened this Oct 14, 2014
@dannydorfel
Copy link
Contributor

Could you please create a pull request for this, together with an updated unit test?

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

No branches or pull requests

2 participants