Skip to content

add shift-click to bypass issued check#2897

Merged
jmcouffin merged 2 commits intopyrevitlabs:developfrom
Wurschdhaud:bypass-issued
Nov 3, 2025
Merged

add shift-click to bypass issued check#2897
jmcouffin merged 2 commits intopyrevitlabs:developfrom
Wurschdhaud:bypass-issued

Conversation

@Wurschdhaud
Copy link
Copy Markdown
Contributor

@Wurschdhaud Wurschdhaud commented Nov 3, 2025

Description

Enhancement for Set Revisions on Sheets


Checklist

Before submitting your pull request, ensure the following requirements are met:

  • Code follows the PEP 8 style guide.
  • Code has been formatted with Black using the command:
    pipenv run black {source_file_or_directory}
  • Changes are tested and verified to work as expected.

Copy link
Copy Markdown
Contributor

@devloai devloai bot left a comment

Choose a reason for hiding this comment

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

PR Summary:

This PR adds a Shift+Click bypass feature to the "Set Revision On Sheets" tool, allowing users to include issued revisions in the selection list. The implementation:

  • Uses pyRevit's built-in EXEC_PARAMS.config_mode mechanism (triggered by Shift+Click)
  • Updates the filter function to conditionally bypass the issued revision check
  • Documents the feature in tooltips across all supported languages (EN, DE, FR, RU)

Review Summary:

The implementation is clean and follows established pyRevit patterns. The code correctly leverages the framework's config_mode property to provide an intuitive Shift+Click bypass mechanism. No logic bugs, security issues, or performance concerns identified. The feature is well-documented in the user-facing tooltips.

Follow-up suggestions:

None needed - the implementation is production-ready as-is.

@jmcouffin jmcouffin merged commit ca44157 into pyrevitlabs:develop Nov 3, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

📦 New work-in-progress (wip) builds are available for 5.2.0.25307+1644-wip

@Wurschdhaud Wurschdhaud deleted the bypass-issued branch November 3, 2025 18:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

📦 New work-in-progress (wip) builds are available for 5.2.0.25307+1935-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

📦 New work-in-progress (wip) builds are available for 5.2.0.25307+2011-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

📦 New work-in-progress (wip) builds are available for 5.2.0.25307+2045-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 3, 2025

📦 New work-in-progress (wip) builds are available for 5.3.0.25307+2237-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 4, 2025

📦 New work-in-progress (wip) builds are available for 5.3.0.25308+0721-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 4, 2025

📦 New public release are available for 5.3.0.25307+2146

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants