Merge pull request #7242 from p12tic/www-add-missing-observer

www: Add missing observer usages in several places
buildbot · Dec 4, 2023 · 2e7eac7 · 2e7eac7
2 parents 6a1b1d4 + c9c4246
commit 2e7eac7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/www/react-base/src/components/BuildSticker/BuildSticker.tsx b/www/react-base/src/components/BuildSticker/BuildSticker.tsx
@@ -15,7 +15,8 @@
   Copyright Buildbot Team Members
 */
 
-import './BuildSticker.scss'
+import './BuildSticker.scss';
+import {observer} from "mobx-react";
 import {Link} from "react-router-dom";
 import { Card } from 'react-bootstrap';
 import {Build, Builder, results2class, results2text} from "buildbot-data-js";
@@ -26,7 +27,7 @@ type BuildStickerProps = {
   builder: Builder;
 }
 
-export const BuildSticker = ({build, builder}: BuildStickerProps) => {
+export const BuildSticker = observer(({build, builder}: BuildStickerProps) => {
   const now = useCurrentTime();
 
   return (
@@ -49,4 +50,4 @@ export const BuildSticker = ({build, builder}: BuildStickerProps) => {
       </Card.Body>
     </Card>
   );
-}
+});
diff --git a/www/react-ui/src/components/ChangeDetails/ChangeDetails.tsx b/www/react-ui/src/components/ChangeDetails/ChangeDetails.tsx
@@ -17,6 +17,7 @@
 
 import './ChangeDetails.scss';
 import {useState} from "react";
+import {observer} from "mobx-react";
 import {OverlayTrigger, Popover, Table} from "react-bootstrap";
 import {Change, parseChangeAuthorNameAndEmail} from "buildbot-data-js";
 import {dateFormat, durationFromNowFormat, useCurrentTime} from "../../util/Moment";
@@ -30,7 +31,7 @@ type ChangeDetailsProps = {
   setShowDetails: (show: boolean) => void;
 }
 
-export const ChangeDetails = ({change, compact, showDetails, setShowDetails}: ChangeDetailsProps) => {
+export const ChangeDetails = observer(({change, compact, showDetails, setShowDetails}: ChangeDetailsProps) => {
   const now = useCurrentTime();
   const [showProps, setShowProps] = useState(false);
 
@@ -137,4 +138,4 @@ export const ChangeDetails = ({change, compact, showDetails, setShowDetails}: Ch
       {showDetails ? renderChangeDetails() : <></>}
     </div>
   );
-}
+});