Skip to content

Remove instructionsEnabled, optionsExplanatoryTextEnabled and seeAlsoEnabled#1457

Merged
demiankatz merged 2 commits intoUniversalViewer:devfrom
Saira-A:remove-options
Jun 23, 2025
Merged

Remove instructionsEnabled, optionsExplanatoryTextEnabled and seeAlsoEnabled#1457
demiankatz merged 2 commits intoUniversalViewer:devfrom
Saira-A:remove-options

Conversation

@Saira-A
Copy link
Copy Markdown
Contributor

@Saira-A Saira-A commented Jun 20, 2025

This removes the instructionsEnabled and optionsExplanatoryTextEnabled options from the configuration files as they are not used and it's unclear what they were intended for

Also removes seeAlsoEnabled, which has a use but is orphaned - could create an issue to make this into a working feature

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
universalviewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 23, 2025 10:08am

@Saira-A Saira-A changed the title Remove instructionsEnabled and optionsExplanatoryTextEnabled Remove instructionsEnabled, optionsExplanatoryTextEnabled and seeAlsoEnabled Jun 23, 2025
Copy link
Copy Markdown
Contributor

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

All looks good/makes sense to me. Thanks, @Saira-A!

@demiankatz demiankatz merged commit 8c37de8 into UniversalViewer:dev Jun 23, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from IN TESTING to COMPLETED in DEV EX Community Sprint May-July 2025 Jun 23, 2025
Saira-A added a commit that referenced this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: COMPLETED

Development

Successfully merging this pull request may close these issues.

2 participants