conveyal / analysis-backend

Server component of Conveyal Analysis
http://conveyal.com/analysis
MIT License
23 stars 12 forks source link

Backward compatibility for CoAXs #158

Closed ansoncfit closed 6 years ago

ansoncfit commented 6 years ago

Reviving this old endpoint allows users who hit analysis-backend with their own front-ends (e.g. CoAXs) to continue using the same requests to access previously uploaded opportunity datasets.

codecov-io commented 6 years ago

Codecov Report

Merging #158 into set-od will increase coverage by <.01%. The diff coverage is 20%.

Impacted file tree graph

@@             Coverage Diff              @@
##             set-od     #158      +/-   ##
============================================
+ Coverage     19.43%   19.43%   +<.01%     
  Complexity       75       75              
============================================
  Files            58       58              
  Lines          2146     2151       +5     
  Branches        190      190              
============================================
+ Hits            417      418       +1     
- Misses         1710     1714       +4     
  Partials         19       19
Impacted Files Coverage Δ Complexity Δ
...taui/controllers/OpportunityDatasetController.java 8.86% <20%> (+0.24%) 3 <0> (ø) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a618394...ec2123a. Read the comment docs.

trevorgerhardt commented 6 years ago

Ah, just pushed a branch with a similar change. I don't think this will work, the old endpoint did not have /region in the path.

ansoncfit commented 6 years ago

Superseded by #159