Specify explicitly that nullable IDs can be filtered on

[eldering]

Also explicitly mention that filtering only needs to be supported on properties listed in the access endpoint.

Adresses the "on a related note" in #193 (comment)

[eldering]

This probably needs to be rebased after #193 is merged.

[deboer-tim]

TBH I don't see the need for this change, to me ID and ID? have the same type. I.e. being optional doesn't change the type. I am not opposed to it if that's confusing someone else though.

The reference to /access actually confuses me though - isn't it obvious enough that you can't filter by something you can't see or doesn't exist? I'm not sure what we're trying to clarify here, but maybe there's a simpler way to say it.

Seeing this also reminded me of the 'except the id property'. I know it isn't being changed here, but I'm not sure if we were trying to save someone work to implement, and it isn't a great way to do it, but it is an odd restriction to make.

[eldering]

TBH I don't see the need for this change, to me ID and ID? have the same type. I.e. being optional doesn't change the type. I am not opposed to it if that's confusing someone else though.

The reference to /access actually confuses me though - isn't it obvious enough that you can't filter by something you can't see or doesn't exist? I'm not sure what we're trying to clarify here, but maybe there's a simpler way to say it.

I think I mostly agree, although this addition would make it explicit that if the property is in /access then you should be able to filter on it, even if in the endpoint itself the property is not explicitly visible because it has (almost) only null values. OTOH, then filtering is a bit pointless...

Seeing this also reminded me of the 'except the id property'. I know it isn't being changed here, but I'm not sure if we were trying to save someone work to implement, and it isn't a great way to do it, but it is an odd restriction to make.

Although it's an exception, I think this makes sense, since you can already "filter" on id by querying for that specific element instead of the whole collection endpoint.

All in all, I'm also fine with just dropping this PR.