-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Children issues fail to render if "ETA" is used instead of "eta" #352
Comments
@BigLep Can you share the link to the issue you're trying to render? |
I created a couple of issues to test this and it seems to be working fine for both variations (
|
@BigLep I recently fixed an issue where our regex would fail to match eta if there were a space prior to "eta:", e.g. " eta:" I tested this on an issue where i had no intentional space in front of "eta:" but for some reason the data we got in our eta parsing method had a space. If you share the issue i can dive into this further or confirm it's resolved. |
FYI that we have a case-insensitive regex grabbing the ETA value at Line 21 in 3dc6509
However, we don't currently have any tests confirming an uppercase ETA: Starmap/tests/unit/parser.test.ts Line 6 in 3dc6509
Action items:
|
My child issues didn't load if I had "ETA" capitalized (even though that's what is in the readme/docs). It did render when I switched to lower case. Can we have case-insensitive matches?
The text was updated successfully, but these errors were encountered: