Closed little-dude closed 8 years ago
Yes, we do have an externalID
Your PR looks good to me. You should launch unit tests locally to ensure everything is working.
@t00f this is ready, once you reviewed the commit that fixes the tests. I'll make a release when this is merged.
Looks good ! Thanks @little-dude
@t00f I think the test is wrong here: we do have an
externalID
attribute, don't we ?