Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Removed restangular from CoordinateService#3609

Merged
mitchell852 merged 2 commits into
apache:masterfrom
ocket8888:removing-restangular-CoordinateService
Jul 26, 2019
Merged

Removed restangular from CoordinateService#3609
mitchell852 merged 2 commits into
apache:masterfrom
ocket8888:removing-restangular-CoordinateService

Conversation

@ocket8888
Copy link
Copy Markdown
Contributor

What does this PR (Pull Request) do?

Removes a dependency on Restangular from the "CoordinateService"

Which Traffic Control components are affected by this PR?

  • Traffic Portal

Traffic Portal dependencies are not individually documented, and so no documentation changes are necessary.

What is the best way to verify this PR?

No functionality should have changed (except that errors will now be logged instead of ignored in many cases), so the existing tests should all pass.

Jeremy: actually, the only way to truly verify that the functionality did not change is to:

  1. run the UI tests (which is a very small subset of all TP functionality)
  2. manually verify the functionality associated with ALL the services functions that changed just to be sure all is well.

The following criteria are ALL met by this PR

  • This PR includes tests OR I have explained why tests are unnecessary
  • I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update is not necessary
  • This PR includes any and all required license headers
  • This PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

@dangogh dangogh added Traffic Portal v1 related to Traffic Portal version 1 tech debt rework due to choosing easy/limited solution and removed tech debt rework due to choosing easy/limited solution labels May 28, 2019
@ocket8888 ocket8888 force-pushed the removing-restangular-CoordinateService branch from c246720 to 5159363 Compare June 3, 2019 14:13
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Jun 3, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3806/
Test PASSed.

@mitchell852 mitchell852 self-assigned this Jun 17, 2019
Copy link
Copy Markdown
Member

@mitchell852 mitchell852 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UI tests pass
Manual coordinate tests pass

@ocket8888 ocket8888 force-pushed the removing-restangular-CoordinateService branch from 5159363 to 032606f Compare July 23, 2019 15:51
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Jul 24, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4004/
Test PASSed.

@mitchell852 mitchell852 merged commit d1cc708 into apache:master Jul 26, 2019
@ocket8888 ocket8888 deleted the removing-restangular-CoordinateService branch July 26, 2019 22:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

tech debt rework due to choosing easy/limited solution Traffic Portal v1 related to Traffic Portal version 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants