Skip to content

XXXX taking over for Vamshi - Update criteria for Bios ready to post to be based on Total Valid Weekly Summaries #3529

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.

Sign up for GitHub

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

Closed
guthaVamshi wants to merge 6 commits into from
+1,055 −767
Closed

XXXX taking over for Vamshi - Update criteria for Bios ready to post to be based on Total Valid Weekly Summaries #3529

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
e997933
updated weeklysummarycount
guthaVamshi Jul 9, 2025
eeda43d
Fix Bio Status filter to use weeklySummariesCount instead of daysInTeam
kanishkagarwal6101 Oct 24, 2025
e8c6a10
fix: convert IssueDashboard.css to module CSS
kanishkagarwal6101 Dec 11, 2025
fa872e7
fix: prettier formatting in IssueDashboard
kanishkagarwal6101 Dec 11, 2025
35b8dfe
fix: resolve SonarQube code quality issues
kanishkagarwal6101 Dec 11, 2025
b4853f3
fix: update Bio Status filter to use weeklySummariesCount >= 8 instea…
kanishkagarwal6101 Dec 16, 2025
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
3 changes: 2 additions & 1 deletion src/components/WeeklySummariesReport/BioFunction.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ function BioFunction(props) {

const [bioStatus, setBioStatus] = useState(bioPosted);

const isMeetCriteria = totalTangibleHrs > 80 && daysInTeam > 60 && bioPosted !== 'posted';
const isMeetCriteria =
totalTangibleHrs > 80 && summary?.weeklySummariesCount >= 8 && bioPosted !== 'posted';
const style = {
color: textColors[summary?.weeklySummaryOption] || textColors.Default,
};
Expand Down
2 changes: 1 addition & 1 deletion src/components/WeeklySummariesReport/FormattedReport.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,7 @@ function ReportDetails({
const isMeetCriteria =
canSeeBioHighlight &&
summary.totalTangibleHrs > 80 &&
summary.daysInTeam > 60 &&
summary.weeklySummariesCount >= 8 &&
summary.bioPosted !== 'posted';

useEffect(() => {
Expand Down
4 changes: 3 additions & 1 deletion src/components/WeeklySummariesReport/WeeklySummariesReport.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,9 @@ export class WeeklySummariesReport extends Component {
const hoursLogged = (summary.totalSeconds[navItems.indexOf(activeTab)] || 0) / 3600;

const isMeetCriteria =
summary.totalTangibleHrs > 80 && summary.daysInTeam > 60 && summary.bioPosted !== 'posted';
summary.totalTangibleHrs > 80 &&
summary.weeklySummariesCount >= 8 &&
summary.bioPosted !== 'posted';

const isBio = !selectedBioStatus || isMeetCriteria;

Expand Down