Skip to content

Update namespaceRegex to include action #3161

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.

Sign up for GitHub

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

Jump to bottom
Closed
miketaylr opened this issue Jan 15, 2020 · 2 comments
Closed

Update namespaceRegex to include action #3161

miketaylr opened this issue Jan 15, 2020 · 2 comments
Assignees
@karlcow
Labels
lang: JS prio: good first bug

Comments

@miketaylr
Copy link
Member

We have some action-foo prefixed labels, https://github.com/webcompat/web-bugs/labels?utf8=%E2%9C%93&q=action

We should update the following Regexp to account for that:

webcompat.com/webcompat/static/js/lib/models/label-list.js

Line 27 in d56cd78

this.set("namespaceRegex", /(browser|closed|os|priority|status)-(.+)/i);

@miketaylr miketaylr mentioned this issue Jan 15, 2020
Merged
@karlcow
Copy link
Member

karlcow commented Jan 15, 2020

@miketaylr This requires to fix #2830 too.
Because right now we are still not getting the full list of labels.

@miketaylr
Copy link
Member Author

Solving #2830 is not a good first bug, but agreed.

karlcow added a commit to karlcow/webcompat.com that referenced this issue Jan 15, 2021
@karlcow
issue webcompat#3161 - Adds action keyword for labels regex
9bb72ac
@karlcow karlcow mentioned this issue Jan 15, 2021
Merged
@karlcow karlcow self-assigned this Jan 15, 2021
@ksy36 ksy36 closed this as completed in d8d937a Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@karlcow karlcow

Labels
lang: JS prio: good first bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@miketaylr @karlcow