Add verification of pickupServicePointId

Description

When a request is created or updated, and contains a pickupServicePointId, we should check this to make sure that:

  • The id corresponds to a valid service point (should be covered by CIRC-151)

  • The corresponding service point is a valid pickup location

Environment

None

Potential Workaround

None

Checklist

hide

TestRail: Results

Activity

Show:

Marc Johnson November 19, 2018 at 11:56 AM

Sure, ping me questions about that (or point me at code if that is easier)

Kurt Nordstrom November 19, 2018 at 11:49 AM

I think I'm very close to having this wrapped, but I discovered some things related to mod-circ's internal error handling mechanisms that I need to chat with about to make certain I'm utilizing them correctly.

Jakub Skoczen November 19, 2018 at 10:00 AM

what is the status for this issue?

Done

Details

Assignee

Reporter

Priority

Fix versions

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created October 25, 2018 at 2:54 PM
Updated November 22, 2018 at 12:44 PM
Resolved November 22, 2018 at 12:44 PM
TestRail: Cases
TestRail: Runs