Skip to content

feat(reporting): Add frontend scan export buttons and print layout ov…#985

Open
Rafiaminhaj wants to merge 5 commits into
utksh1:mainfrom
Rafiaminhaj:feature/report-exporter-frontend
Open

feat(reporting): Add frontend scan export buttons and print layout ov…#985
Rafiaminhaj wants to merge 5 commits into
utksh1:mainfrom
Rafiaminhaj:feature/report-exporter-frontend

Conversation

@Rafiaminhaj

Copy link
Copy Markdown
Collaborator

…errides

Description

Related Issues

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Requesting changes because this frontend export/print PR is not currently passing frontend-checks and changes a user-facing workflow. Please get the frontend check green and keep the export/print behavior covered with focused tests or screenshots so we can verify it does not regress the TaskDetails layout.

@utksh1 utksh1 added type:feature Feature work category bonus label area:frontend Frontend React/UI work area:backend Backend API, database, or service work level:intermediate 35 pts difficulty label for moderate contributor PRs labels Jun 23, 2026
@Rafiaminhaj Rafiaminhaj force-pushed the feature/report-exporter-frontend branch from 30f4844 to 46b0ea4 Compare June 23, 2026 07:20
@Rafiaminhaj Rafiaminhaj force-pushed the feature/report-exporter-frontend branch from 46b0ea4 to 2bb91ec Compare June 23, 2026 07:55
@Rafiaminhaj Rafiaminhaj force-pushed the feature/report-exporter-frontend branch from 1bab8e6 to 112db80 Compare June 23, 2026 10:40
@utksh1

utksh1 commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Closing due to unresolved review feedback.

@utksh1 utksh1 closed this Jun 24, 2026
@utksh1 utksh1 added the gssoc:invalid Admin validation: invalid for GSSoC scoring label Jun 24, 2026
@utksh1 utksh1 reopened this Jun 25, 2026
@utksh1 utksh1 removed the gssoc:invalid Admin validation: invalid for GSSoC scoring label Jun 25, 2026
@utksh1

utksh1 commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Closing this PR — despite recent commits, the previously requested changes have not been adequately addressed. The PR remains bloated with unrelated changes / unresolved review concerns. Please open a fresh, focused PR if you'd like to revisit this work.

@utksh1 utksh1 closed this Jun 25, 2026
@utksh1 utksh1 added the gssoc:invalid Admin validation: invalid for GSSoC scoring label Jun 25, 2026
@utksh1 utksh1 reopened this Jun 25, 2026
@utksh1 utksh1 added gssoc:invalid Admin validation: invalid for GSSoC scoring and removed gssoc:invalid Admin validation: invalid for GSSoC scoring labels Jun 25, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for this contribution! I've reviewed the proposal and it looks like a valid addition. However, this branch currently has Git merge conflicts with main. Could you please pull the latest changes from upstream, resolve the conflicts, and push the rebased branch so we can continue the review process?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:frontend Frontend React/UI work level:intermediate 35 pts difficulty label for moderate contributor PRs type:feature Feature work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants