Add triage statuses#188
Conversation
ksy36
force-pushed
the
triage_statuses
branch
3 times, most recently
from
05368b4 to
5f08494
Compare
| triage_status='worksforme', | ||
| triaged_at=now | ||
| ) | ||
|
|
There was a problem hiding this comment.
There are 3 migrations:
- Adds
triage_statusandtriaged_atfields - All existing triaged issues are moved to
worksformewithtriaged_atset to previoushide_untilminus 4 weeks hide_untilfield is removed
| now = timezone.now() | ||
| Bucket.objects.filter(hide_until__lte=now).update(hide_until=None) | ||
|
|
||
|
|
There was a problem hiding this comment.
I've removed this existing unhide_buckets command and adding a new command that would reset triage status based on spikes in a separate PR #194
| </script> | ||
|
|
||
| <style scoped> | ||
| .triage-bucket-dropdown { |
There was a problem hiding this comment.
Not a blocker for this PR, but I'm not a massive fan of us having all this CSS specific to one dropdown. As we accumulate more components we're just adding lots of CSS for each one and it might be better to try to refactor a bit.
There was a problem hiding this comment.
True, I can probably just use bootstrap's dropdown since we're already including it:) I'll change it in a separate PR
ksy36
force-pushed
the
triage_statuses
branch
2 times, most recently
from
8b3643c to
d84c22d
Compare
| } | ||
|
|
||
| function fetch_api_data() { | ||
| fetch(notify_api_url+'?max='+notify_fetch_count, {credentials: 'same-origin'}) |
There was a problem hiding this comment.
I've noticed this request if failing on every page since we're no longer using notifications, so removing it
2 participants
Changes "Mark triaged until " to "Mark triaged with a status"