' consistent with other api endpoints
Closed, MigratedEdits Locked
Actions

Assigned To

Authored By

	jbertran
	Aug 22 2016, 3:55 PM

Description

More work needs to be done on that endpoint (which I created based on my needs for the calendar UI). Namely:

Change the route to /api/1/origin/<int>/visits/ for consistency
Return more interesting and complete data than just the bare timestamps of the visits
Add the origin visit status information to what we display, and display that information on the graph, when that information becomes available via the storage API (see T533)

Revisions and Commits

rDWAPPS Web applications
	rDWAPPS7422120ccc49 /api/1/origin/<origin_id>/visits/<visit_id>: Open endpoint

Related Objects
Search...

Status	Assigned	Task
Migrated	gitlab-migration	T533 Allow loaders to register partial state (meta task)
Migrated	gitlab-migration	T541 Make the origin visit '/api/1/stat/visits/<int:origin_id>/' consistent with other api endpoints
Migrated	gitlab-migration	T559 Open /api/1/origin/<origin_id>/visits/<visit_id>/ endpoint

Event Timeline

jbertran created this task.Aug 22 2016, 3:55 PM

jbertran created this object in space S1 Public.

jbertran added a project: Web app.

jbertran added a parent task: T533: Allow loaders to register partial state (meta task).

More suggestions:

add start date/end date filters
add a possibility to limit the number of visits returned (in the forseeable future, this wouldn't be hit anyway)

ardumont mentioned this in rDWAPPSf90f7ae6544c: Make origin_visit api endpoint consistent with existing api.Sep 2 2016, 1:33 PM

ardumont renamed this task from Review the origin visit endpoint to Make the origin visit '/api/1/stat/visits/<int:origin_id>/' consistent with other api endpoints.Sep 2 2016, 2:55 PM

ardumont updated the task description. (Show Details)

ardumont created subtask T559: Open /api/1/origin/<origin_id>/visits/<visit_id>/ endpoint.Sep 2 2016, 2:59 PM

ardumont claimed this task.Sep 2 2016, 3:17 PM

ardumont closed this task as Resolved by committing rDWAPPS7422120ccc49: /api/1/origin/<origin_id>/visits/<visit_id>: Open endpoint.Sep 2 2016, 3:24 PM

ardumont added a commit: rDWAPPS7422120ccc49: /api/1/origin/<origin_id>/visits/<visit_id>: Open endpoint.

ardumont closed subtask T559: Open /api/1/origin/<origin_id>/visits/<visit_id>/ endpoint as Resolved.Sep 5 2016, 12:28 PM

ardumont mentioned this in R65:f90f7ae6544c: Make origin_visit api endpoint consistent with existing api.Feb 23 2019, 1:55 AM