Skip to content

Conversation

camd
Copy link
Collaborator

@camd camd commented Sep 26, 2025

I have gone over Treeherder, perfherder, Intermittent Failures View and Push Health. They all look good to me. But there is a lot, so I may have missed something.

@camd camd self-assigned this Sep 26, 2025
@camd camd force-pushed the camd/react-bootstrap-fixups branch from b3c6aa6 to e75521b Compare September 26, 2025 04:05
@camd camd force-pushed the camd/react-bootstrap-fixups branch 4 times, most recently from a93f2cf to 90e3965 Compare October 5, 2025 16:54
@camd camd marked this pull request as ready for review October 6, 2025 14:59
@camd camd force-pushed the camd/react-bootstrap-fixups branch 2 times, most recently from b5072a5 to 8eba2f1 Compare October 6, 2025 16:58
@codecov-commenter
Copy link

codecov-commenter commented Oct 6, 2025

Codecov Report

❌ Patch coverage is 57.61905% with 178 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.51%. Comparing base (4ef25ce) to head (87b9115).

Files with missing lines Patch % Lines
...i/job-view/details/shared/ResponsiveNavigation.jsx 0.00% 58 Missing ⚠️
ui/job-view/details/tabs/TabsPanel.jsx 72.80% 34 Missing ⚠️
ui/push-health/ClassificationGroup.jsx 41.17% 10 Missing ⚠️
ui/shared/BugFiler.jsx 52.63% 9 Missing ⚠️
ui/infra-compare/InfraCompareTableRow.jsx 0.00% 8 Missing ⚠️
ui/push-health/Test.jsx 20.00% 8 Missing ⚠️
ui/perfherder/alerts/StatusDropdown.jsx 40.00% 6 Missing ⚠️
ui/job-view/pushes/Push.jsx 42.85% 3 Missing and 1 partial ⚠️
ui/perfherder/shared/Pagination.jsx 55.55% 4 Missing ⚠️
ui/perfherder/compare/SelectorCard.jsx 72.72% 3 Missing ⚠️
... and 25 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8996      +/-   ##
==========================================
- Coverage   77.73%   77.51%   -0.22%     
==========================================
  Files         581      582       +1     
  Lines       29710    29876     +166     
  Branches     3577     3664      +87     
==========================================
+ Hits        23095    23159      +64     
- Misses       6455     6550      +95     
- Partials      160      167       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

import React from 'react';
import PropTypes from 'prop-types';
import { Badge, Button, FormGroup, Input } from 'reactstrap';
import { Badge, Button, Form } from 'react-bootstrap';
Copy link
Collaborator

@beatrice-acasandrei beatrice-acasandrei Oct 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2025-10-07 at 16 11 38

Locally I see some issues with this component.


## Overview

This document tracks differences found between the local development version of Treeherder (<http://localhost:5001>) and the staging version (<https://treeherder.allizom.org>) when comparing the Perfherder interface.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logging in locally is no longer working for http://localhost:5001/

@camd camd force-pushed the camd/react-bootstrap-fixups branch from 8eba2f1 to 87b9115 Compare October 8, 2025 20:50
@Archaeopteryx
Copy link
Collaborator

Some observations:

  1. All the links are underscored. Can we get the former behavior back? That's a change in bootstrap v5.
  2. The login was not working. The latest commit added to this branch allows to log in but still shows an error for Taskcluster because the new client needs to be allowed and the PR for that is still pending review.
  3. In the toolbar at the top while logged in, the bell icon has a dropdown which is not available without the patch - only one options should be available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants