IAU-ADES / ADES-Master

ADES implementation based on a master XML file
26 stars 7 forks source link

Expand allowed character set for obsSubID #32

Closed stevechesley closed 7 months ago

stevechesley commented 9 months ago

There is a request to allow hyphens in obsSubID. This seems reasonable since obsSubID is generally only used by the submitter, and so the submitter should choose the form of the string. At present obsSubIDis AlphaNumeric Type (regex: [A-Za-z0-9_]*). Consider changing this to StringType (regex: [^|]*[^|\s][^|]*), which allows any string that does not include pipes but does include at least one non-white space character.

I will give this a week or two to allow for comments and then implement.

stevechesley commented 7 months ago

Resolved with PR #37