blender/blender-addons
19
43
Fork 96
Code Issues 634 Pull Requests 27 Projects 1 Releases Wiki Activity

Cleanup: import-brush-set-replace-hardcoded-file-extensions #104802

Merged
Campbell Barton merged 6 commits from DanielGrauer/blender-addons:import-brush-set-replace-hardcoded-file-extensions into main 2023-08-22 03:17:51 +02:00
Conversation 3 Commits 6 Files Changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5c84e61055 - Show all commits

2
io_import_BrushSet.py
View File

@ -16,7 +16,7 @@ bl_info = {
"description": "Imports all image files from a folder.",
"warning": '', # used for warning icon and text in addons panel
"doc_url": "http://wiki.blender.org/index.php/Extensions:2.5/Py/Scripts/Import-Export/BrushSet",
"tracker_url": "https://projects.blender.org/blender/blender-addons/src/branch/main/io_import_BrushSet.py",
"tracker_url": "https://developer.blender.org/maniphest/task/edit/form/2/",
Campbell Barton commented 2023-08-21 02:23:29 +02:00
Outdated
Review
Copy Link

This is the URL to the file in GIT, this is not the tracker URL.

Please don't make unrelated changes, it complicates and slows down patch review.

This is the URL to the file in GIT, this is not the tracker URL. Please don't make unrelated changes, it complicates and slows down patch review.
Daniel Grauer commented 2023-08-21 08:30:45 +02:00
Outdated
Review
Copy Link

What would then be the link to use?

What would then be the link to use?
Daniel Grauer commented 2023-08-21 19:42:18 +02:00
Outdated
Review
Copy Link

I reverted the tracker url change.

I reverted the tracker url change.
Campbell Barton commented 2023-08-22 03:10:27 +02:00
Outdated
Review
Copy Link

Thanks, we should really go over & correct every tracker_url for add-ons in one pass.
Typically this is to report bugs, so linking to the add-on report field makes most sense, we might be able to include arguments to set the add-on name so any reports include the proper add-on.

It's outside the scope of this patch though.

Thanks, we should really go over & correct every tracker_url for add-ons in one pass. Typically this is to report bugs, so linking to the add-on report field makes most sense, we might be able to include arguments to set the add-on name so any reports include the proper add-on. It's outside the scope of this patch though.
"category": "Import-Export",
}