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

Removed restangular from TypeService#3597

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

Removed restangular from TypeService#3597
mitchell852 merged 2 commits into
apache:masterfrom
ocket8888:removing-restangular-TypeService

Conversation

@ocket8888
Copy link
Copy Markdown
Contributor

What does this PR (Pull Request) do?

Removes a dependency on Restangular from the "TypeService"

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 the Traffic Portal v1 related to Traffic Portal version 1 label May 28, 2019
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented May 28, 2019

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

@dangogh dangogh added tech debt rework due to choosing easy/limited solution and removed tech debt rework due to choosing easy/limited solution labels May 30, 2019
@ocket8888 ocket8888 force-pushed the removing-restangular-TypeService branch from 35d6a1b to f7cd06a Compare June 3, 2019 14:22
@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/3813/
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 types tests pass

@ocket8888 ocket8888 force-pushed the removing-restangular-TypeService branch from f7cd06a to 535853f Compare July 23, 2019 16:01
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Jul 25, 2019

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

@mitchell852 mitchell852 merged commit 5da4a5b into apache:master Jul 26, 2019
@ocket8888 ocket8888 deleted the removing-restangular-TypeService branch July 26, 2019 22:52
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