Carlos Alvarez
Carlos Alvarez
Yeah that looks good, and I submitted a back end fix so when the tagName is `Tag` we don't send it. Is there anything else that looks like it's not...
@parasharrajat the videos don't show any of the tests, can you please update them to show the following? > 1. Violations should be visible when viewing the request on each...
> * If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack...
The onyx data changes reflect in the UI in the form of violations
Violations are showing as you can see in some of the screenshots in the PR description. We should test that they're all showing on the right fields when applicable, and...
@parasharrajat I added [email protected] to the violation beta, but let me know if you want me to add any other test accounts
Given this is push notification, I bet this is a back end issue that needs to be fixed internally
Sorry I'm still a bit behind and haven't gotten to this, but it's on my list
Looking more into this, this isn't a push notification, but a browser local notification, so I think it can be handled externally. There might be an inconsistency between how we...
@johncschuster I think External and Help Wanted are the labels to apply to get this through the normal bug process, but correct me if I'm wrong