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

Fix #1033: URP endpoint for beta now points to production. #1034

Merged
merged 1 commit into from Apr 8, 2019

Conversation

@jhreis
Copy link
Contributor

jhreis commented Apr 8, 2019

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • My patch or PR title has a standard commit message that looks like Fix #123: This fixes the shattered coffee cup! (or No Bug: <message> if no relevant ticket)
  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()
  • New files have MPL-2.0 license header.

Test Plan:

Screenshots:

Reviewer Checklist:

  • PR is linked to an issue via Zenhub.
  • Issues are assigned to at least one epic.
  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable)
@jhreis jhreis changed the title Fix #1033: URP endpoint for beta now points to production. [Blocked] Fix #1033: URP endpoint for beta now points to production. Apr 8, 2019
@jhreis jhreis changed the title [Blocked] Fix #1033: URP endpoint for beta now points to production. Fix #1033: URP endpoint for beta now points to production. Apr 8, 2019
Copy link
Contributor

danishjafri88 left a comment

looks good.

@danishjafri88 danishjafri88 merged commit fb0e544 into development Apr 8, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@jhreis jhreis deleted the urp_endpoint_beta_change branch Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.