Feat session info converters - #880
Open
Creylay wants to merge 2 commits into
Open
Conversation
…nce SessionInfoContent with converter metadata
Creylay
added this pull request to stack #879
September 11, 2026 17:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixed the session info modal getting cut off with no scroll when its content was too long, and added an "Applied Converters" section to it (same style as input/output columns).
Type of Change
Check all that apply like this [x]:
Changes (by file)
shared/InfoModal.jsx: the modal's outer container now caps its height at 85vh and lays out as a flex column, the header no longer shrinks, and the body scrolls on its own (overflowY: auto,flex: 1,minHeight: 0), so long content no longer gets cut off with no way to reach it.models/SessionInfoContent.jsx: added an "Applied Converters" section between "Session Configuration" and "Metadata", shown only when the session has preprocessing steps. Each step is rendered as aParamInfoBox(same style as input/output columns), with its display name and scope resolved through the existingbuildStepDisplayNamesandbuildColumnKeysAndTypeshelpers, so a converter's name and its referenced columns (raw or chained from an earlier step's output group) show exactly as they do in the rest of the session wizard.utils/i18n/locales/{en,es,de,pt,zh}/models.json: added theappliedConverterslabel key used by the new section.Testing (optional)