Skip to content

Update "Generate Revision Report" Tool to give more control over whic…#2667

Merged
jmcouffin merged 3 commits intopyrevitlabs:developfrom
tay0thman:patch-20
May 16, 2025
Merged

Update "Generate Revision Report" Tool to give more control over whic…#2667
jmcouffin merged 3 commits intopyrevitlabs:developfrom
tay0thman:patch-20

Conversation

@tay0thman
Copy link
Copy Markdown
Contributor

@tay0thman tay0thman commented May 15, 2025

Tool to give more control on which Revisions to be reported

This is a minor tweak requested by people around me where the Contractor asked for a quick report on what changed in a specific revision.

I replaced the FilteredElementCollector with forms.select_revisions which will give the users more flexibility and control over their report.

…h Revisions to be reported

This is a minor tweak requested by people around me where the Contractor asked for a quick report on what changed in a specific revision.

I replaced the FilteredElementCollector with forms.select_revisions which will give the users more flexibility and control over their report.
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 enhances the "Generate Revision Report" tool by replacing the FilteredElementCollector with forms.select_revisions. This change gives users more flexibility and control over which revisions to include in their report, making it easier to generate targeted reports for specific revisions as requested by contractors.

Review Summary:

The change is straightforward and improves user control over the report generation process. I've identified a couple of issues that need to be addressed before merging: a missing import statement and a potential edge case when users cancel the selection dialog. These fixes will ensure the tool works reliably in all scenarios.

Please feel free to provide feedback on this review - I'm happy to clarify any points or discuss alternative approaches.

Follow-up suggestions:

@devloai fix the identified issues in a follow-up commit

tay0thman added 2 commits May 15, 2025 12:06
Add forms import
Added script exit if no selection
@jmcouffin jmcouffin merged commit bab84c4 into pyrevitlabs:develop May 16, 2025
@jmcouffin
Copy link
Copy Markdown
Contributor

Thanks again @tay0thman

@jmcouffin jmcouffin added the Tools Issues related to pyRevit commands [subsystem] label May 16, 2025
@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0631-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0639-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25140+1615-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25141+1904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25143+0904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25146+0559-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1012-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1943-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 2, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25153+1459-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0858-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 3, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0920-wip

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 4, 2025

📦 New work-in-progress (wip) builds are available for 5.1.0.25155+0904-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1125-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1306-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25162+2030-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+0700-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1347-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1420-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1830-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.1.0.25171+0757-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New work-in-progress (wip) builds are available for 5.2.0.25181+1313-wip

@github-actions
Copy link
Copy Markdown
Contributor

📦 New public release are available for 5.2.0.25181+1425

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

Labels

Tools Issues related to pyRevit commands [subsystem]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants