Closed Bankde closed 4 years ago
I have noticed that many REST API endpoints do not contain any extension For example:
constant("API_ENDPOINT_TEAM_MEMBERSHIPS","api/team_memberships")
So I have updated the regex and also add simple unit test.
I also change (?:[\?|/][^"|']{0,}|)) to this (?:[\?|#][^"|']{0,}|)) I don't remember why we need trailing /. I think ? or # make more sense.
(?:[\?|/][^"|']{0,}|))
(?:[\?|#][^"|']{0,}|))
Any comment on this PR ? This PR will fix what it could have missed as in the above example.
I have noticed that many REST API endpoints do not contain any extension For example:
So I have updated the regex and also add simple unit test.
I also change
(?:[\?|/][^"|']{0,}|))
to this(?:[\?|#][^"|']{0,}|))
I don't remember why we need trailing /. I think ? or # make more sense.