Skip to content

Update extensions.json - Add T3LabLite extension details#3208

Merged
jmcouffin merged 5 commits intopyrevitlabs:developfrom
thanhtranarch:patch-1
Apr 5, 2026
Merged

Update extensions.json - Add T3LabLite extension details#3208
jmcouffin merged 5 commits intopyrevitlabs:developfrom
thanhtranarch:patch-1

Conversation

@thanhtranarch
Copy link
Copy Markdown
Contributor

@thanhtranarch thanhtranarch commented Mar 31, 2026

Added T3LabLite extension with details including author, description, and URLs.

Name of your PR

Description

Please provide a brief description of the changes introduced in this pull request. Explain the purpose of these changes and their intended effect on the project.


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.

Related Issues

If applicable, link the issues resolved by this pull request:

  • Resolves #[issue number]

Additional Notes

Include any additional context, screenshots, or considerations for reviewers.


Thank you for contributing to pyRevit! 🎉

Added T3LabLite extension with details including author, description, and URLs.
@devloai
Copy link
Copy Markdown
Contributor

devloai bot commented Mar 31, 2026

Unable to trigger custom agent "Code Reviewer". You have run out of credits 😔
Please upgrade your plan or buy additional credits from the subscription page.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds T3LabLite to pyRevit’s public extension registry (extensions/extensions.json) so it can be discovered and installed/configured through pyRevit’s extension lookup mechanism.

Changes:

  • Register new extension entry for T3LabLite with author, description, and repository/website URLs.

Comment on lines +262 to +264
"author": "Tran Tien Thanh",
"author_profile": "https://www.linkedin.com/in/sunarch7899/",
"url": "https://github.com/thanhtranarch/T3Lab-Lite_RevitTools",
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

author_profile is the key used by the Python-side extension package parser, but the C# PyRevitExtensionDefinition.AuthorProfile getter reads author_url (see dev/pyRevitLabs/pyRevitLabs.PyRevit/PyRevitExtension.cs). With only author_profile present, CLI / C# consumers will likely show an empty author profile for this entry. Consider adding an author_url field (keeping author_profile for backward compatibility) or aligning the key name to what the C# parser expects.

Copilot uses AI. Check for mistakes.
@jmcouffin
Copy link
Copy Markdown
Contributor

The pull request itself looks good.
But your repo structure is wrong.
Your repo's root should contain the content of your actual extension folder.
The readme steps are wrong too.

Also, if there is any code in your extension that allows traffic in or out please point to the relevant bits to save me from having to screen the whole repo.

@thanhtranarch
Copy link
Copy Markdown
Contributor Author

Hi @jmcouffin, thanks for the feedback!
I've updated the PR:
Structure: Updated the structure.
README: Updated the setup steps and added a dedicated Network Traffic section for your review.
Please let me know if you need anything else. Thanks!

"description": "Set of tools for Revit.",
"author": "Tran Tien Thanh",
"author_profile": "https://www.linkedin.com/in/sunarch7899/",
"url": "https://github.com/thanhtranarch/T3Lab-Lite_RevitTools",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like it is the wrong url if I compare to your repo list and recent activity

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

.../T3LabLite.git

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, it should be /T3LabLit.git . I've updated URL and push a new commit. Thank you!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jmcouffin jmcouffin merged commit 4783cca into pyrevitlabs:develop Apr 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

📦 New public release are available for 6.3.0.26095+0830

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.

3 participants