Closed mratford closed 9 months ago
Looks ok to me. I think it would be useful to add tests with at least one example of each anticipated id format, for the benefit of those working on it in future
I've pinched your tests from a previous version.
I've pinched your tests from a previous version.