Skip to content
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

New Metas view, add logo header, new styles, cleanup old cruft #294

Merged
merged 18 commits into from
Feb 27, 2024
Merged
Show file tree
Hide file tree
Changes from 16 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
57 changes: 0 additions & 57 deletions .eslintrc.js

This file was deleted.

55 changes: 55 additions & 0 deletions .eslintrc.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,55 @@
# When adding items to this file please check for effects on sub-directories.
parser: "@typescript-eslint/parser"
parserOptions:
ecmaVersion: 2022
ecmaFeatures:
jsx: true
sourceType: module
env:
node: true
browser: true
plugins:
- prettier
- import
- react
- react-hooks
- "@typescript-eslint"
extends:
- eslint:recommended
- plugin:react/recommended
- plugin:react-hooks/recommended
- prettier
- plugin:prettier/recommended
rules:
no-empty:
- error
- allowEmptyCatch: true
no-unused-vars: 0
"@typescript-eslint/no-unused-vars": 2
prettier/prettier:
- error
- {}
- usePrettierrc: true
# Why no react/jsx-no-bind?
# See: https://twitter.com/Meligy/status/1069074157151780864
# https://reactjs.org/docs/hooks-faq.html#are-hooks-slow-because-of-creating-functions-in-render
# https://cdb.reacttraining.com/react-inline-functions-and-performance-bdff784f5578
react/prop-types: 0
# TODO: Remove no-deprecated when finished migrating
react/no-deprecated: 0
react/jsx-no-target-blank: 2
react/jsx-no-undef: 2
react/jsx-pascal-case: 2
react/no-access-state-in-setstate: 2
react/no-danger: 2
react/no-did-mount-set-state: 2
react/no-did-update-set-state: 2
react/no-direct-mutation-state: 2
react/no-unknown-property: 0
react-hooks/rules-of-hooks: 2
react-hooks/exhaustive-deps: 1
globals:
globalThis: false
settings:
react:
version: detect
40 changes: 22 additions & 18 deletions .prettierrc
Original file line number Diff line number Diff line change
@@ -1,18 +1,22 @@
{
"arrowParens": "avoid",
"bracketSameLine": true,
"bracketSpacing": true,
"embeddedLanguageFormatting": "auto",
"endOfLine": "lf",
"printWidth": 80,
"proseWrap": "preserve",
"quoteProps": "as-needed",
"semi": true,
"singleQuote": false,
"tabWidth": 2,
"trailingComma": "es5",
"useTabs": false,
"overrides": [
{ "files": [".prettierrc", ".parcelrc"], "options": { "parser": "json" } }
]
}
arrowParens: avoid
bracketSameLine: true
bracketSpacing: true
embeddedLanguageFormatting: auto
endOfLine: lf
printWidth: 80
proseWrap: preserve
quoteProps: as-needed
semi: true
singleQuote: false
tabWidth: 2
trailingComma: es5
useTabs: false
overrides:
- files:
- .prettierrc
options:
parser: yaml
- files:
- .parcelrc
options:
parser: json
21 changes: 1 addition & 20 deletions src/config/people.js
Original file line number Diff line number Diff line change
@@ -1,23 +1,8 @@
export const emails = {
"andrei.br92@gmail.com": "andrei",
"khudson@mozilla.com": "k88hudson",
"usarracini@mozilla.com": "ursula",
"edilee@mozilla.com": "ed",
"elee@mozilla.com": "ed",
"dmose@mozilla.org": "dmose",
"dmosedale@mozilla.com": "dmose",
"msamuel@mozilla.com": "marina",
"rrosario@mozilla.com": "ricky",
"bbell@mozilla.com": "bryan",
"najiang@mozilla.com": "nan",
"csadilek@mozilla.com": "csadilek",
"tspurway@mozilla.com": "tim",
"ahillier@mozilla.com": "adam",
"sdowne@getpocket.com": "scott",
"gsuntop@getpocket.com": "gvn",
"mathijs@getpocket.com": "mathijs",
"pdahiya@mozilla.com": "punam",
"jcarlos@mozilla.com": "jeane",
"ssami@getpocket.com": "sumita",
"emcminn@mozilla.com": "emily",
"shughes@mozilla.com": "aminomancer",
"vtay@mozilla.com": "venetia",
Expand All @@ -26,10 +11,6 @@ export const emails = {
"nsauermann@mozilla.com": "negin",
};

export const ui_emails = ["wkonu@getpocket.com", "abenson@mozilla.com"];

export const qa_emails = ["bnagabandi@getpocket.com"];

export const teams = {
omc: [
"edilee@mozilla.com",
Expand Down
7 changes: 1 addition & 6 deletions src/config/project_settings.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,7 @@
// The bugzilla product bugs are found in
export const BUGZILLA_PRODUCT = "Firefox";
// What components should we check to do triage?
export const BUGZILLA_TRIAGE_COMPONENTS = [
"New Tab Page",
"Pocket",
"Messaging System",
"Nimbus Desktop Client",
];
export const BUGZILLA_TRIAGE_COMPONENTS = ["Messaging System"];
// Components to check when assigned to general triage
export const BUGZILLA_GENERAL_TRIAGE_COMPONENTS = [
{ product: "Firefox", component: "General" },
Expand Down
9 changes: 0 additions & 9 deletions src/config/tags.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,6 @@ export default {
},
label: "has PR",
},
uiwanted: {
label: "uiwanted",
style: {
backgroundColor: "var(--magenta-60)",
color: "white",
border: 0,
fontWeight: "bold",
},
},
"strings needed": {
label: "strings needed",
style: {
Expand Down
14 changes: 12 additions & 2 deletions src/content/components/AboutView/AboutView.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,23 @@ export class AboutView extends React.PureComponent {
return (
<div className={styles.container}>
<h1>About</h1>
Bugzy is a dashboard for tracking specific Bugzilla bugs. It&apos;s
designed to be used by teams that want to track bugs in a specific
iteration, or bugs that are blocking a specific bug.
<p>
<a
href="https://github.com/mozilla/bugzy"
target="_blank"
rel="noopener noreferrer">
Repository
</a>
</p>
<p>
Please file issues at{" "}
<a
href="https://github.com/mozilla/bugzy/issues/new/choose"
target="_blank"
rel="noopener noreferrer">
github.com/mozilla/bugzy
Issues
aminomancer marked this conversation as resolved.
Show resolved Hide resolved
</a>
</p>
<h3>Credits</h3>
Expand Down
3 changes: 2 additions & 1 deletion src/content/components/AboutView/AboutView.module.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
.container {
max-width: 700px;
max-width: 640px;
margin: 0 auto;
padding: 20px 40px;
font-size: 14px;
}
Original file line number Diff line number Diff line change
@@ -1,17 +1,21 @@
.RSTable {
padding: .6em;
padding: 0.6em;
margin: 0 auto;
width: max-content;
aminomancer marked this conversation as resolved.
Show resolved Hide resolved

td {
padding: .6em;
padding: 0.6em;
}

tbody tr {
border-bottom: 1px solid #ddd;
tbody {
font-family: var(--monospace-font);

&:hover {
background: #ddd;
tr {
border-bottom: 1px solid #ddd;

&:hover {
background: #ddd;
}
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,7 @@ export class ActiveRSMessages extends React.PureComponent {
<div className={styles.RSTable}>
<h1>Active Remote Settings Messages</h1>
<h3>
Select RS Bucket:
Select RS Bucket:&nbsp;
<select onChange={this.bucketUpdate} defaultValue="CFR">
{Object.keys(this.state.buckets).map(bkey => (
<option value={bkey} key={bkey}>
Expand Down
Loading