Remove deprecated support for unset hosting service IDs.

Review Request #10652 — Created July 30, 2019 and submitted

Information

Review Board
release-4.0.x
62de7e5...

Reviewers

Review Board 3.0 is gaining support for setting an explicit hosting
service ID, which we previously didn't correctly support when using a
HostingServiceHook. It was defaulting to using the service's name,
which wasn't a valid ID. To fix this, it automatically slugified the
name, turning it into an ID, but this is meant to be temporary,
deprecated behavior for 3.0.

This change removes that behavior, replacing it instead with an error
that's raised if the ID is missing.

It also removes some unused imports from previous deprecation removals.

Unit tests pass.

david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (6a6a16c)
Loading...