Skip to content

Display applied filters in HTML report#64

Merged
ksauraj merged 4 commits intoksauraj:masterfrom
sparshsinghal885:feat/display-filters
Oct 2, 2025
Merged

Display applied filters in HTML report#64
ksauraj merged 4 commits intoksauraj:masterfrom
sparshsinghal885:feat/display-filters

Conversation

@sparshsinghal885
Copy link
Contributor

Description

This PR resolves the issue of generated HTML files lacking context. It introduces a feature to display a summary of all user-selected filters directly at the top of the HTML report. This makes the output files self-documenting and much more useful when shared.

Summary of Changes

  • Filter Tracking: Implemented a dictionary (applied_filters) that is initialized at the start of a counseling session.
  • Data Propagation: This dictionary is passed through the function call stack, with each step adding the relevant user choice (e.g., year, round, institute type).
  • Dynamic Rendering: The display_df_web function now uses this dictionary to dynamically generate an "Applied Filters" list in the HTML output.

Example of New Output

The generated HTML file will now include a section like this:

!

Applied Filters:

  • Counseling Type: JOSAA
  • Year: 2023
  • Round: 6
  • Institute Type: NITs
  • Program: Computer Science and Engineering
  • Institute: All
  • Quota: All India, Home State
  • Seat Type: OPEN, EWS
  • Gender: Gender-Neutral
  • Your Rank (Show colleges with closing rank >): 12424

Adds a new section to the generated HTML output that lists all the filters applied by the user. This makes the reports self-contained and easier to understand and share.
@ksauraj ksauraj self-assigned this Oct 2, 2025
@ksauraj ksauraj added enhancement New feature or request approved ✅ PR approved by maintainer. hacktoberfest hacktoberfest2025 labels Oct 2, 2025
@ksauraj
Copy link
Owner

ksauraj commented Oct 2, 2025

Seems good

@ksauraj ksauraj merged commit 64a1a8e into ksauraj:master Oct 2, 2025
1 check passed
ksauraj added a commit that referenced this pull request Oct 2, 2025
ksauraj added a commit that referenced this pull request Oct 2, 2025
This reverts commit 64a1a8e.
- reverting due to multiple merge conflicts on previous commits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ✅ PR approved by maintainer. enhancement New feature or request hacktoberfest hacktoberfest2025

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants