-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(UI): CNX-9070 update connectors to use new fe2 terminology #186
feat(UI): CNX-9070 update connectors to use new fe2 terminology #186
Conversation
JR-Morgan
commented
Feb 27, 2024
•
edited
Loading
edited
- Updated naming to FE2
- Updated the "Open Stream in Web" button to go to the specific branch + commit, not just the stream (and ofcourse, working with FE1 and FE2 accounts
- Marked a couple functions as deprecated (Code that were not using, or maintaining)
* Increased default branch get to 100 limit, and added as mesh conversion for text + surfaces + metaball * poetry lock * Upgraded typing module * FE2 URL support * Raised exceptions now display to user
@@ -11,19 +11,19 @@ | |||
|
|||
class DeleteCommit(bpy.types.Operator): | |||
""" | |||
Deletes the selected commit from the selected stream. | |||
Permanently deletes the selected version from the selected model. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the change from project
(stream) to model
intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is, I was trying to align with the new api. Where versions are a property of a model, and not of a stream.
But... perhaps we could discuss this more as a team, and align on our naming here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me - I see you've switched to title case for the labels, feel free to ignore earlier comments on lowercase capitalization!
* feat(UI): CNX-9070 update connectors to use new fe2 terminology (#186) * feat: CNX-8705 fe2 ur ls in blender (#182) * Increased default branch get to 100 limit, and added as mesh conversion for text + surfaces + metaball * poetry lock * Upgraded typing module * FE2 URL support * Raised exceptions now display to user * Fixed unused imports * Updated terminology to fe2 * merge from stash * comments * bl_descriptions * bl_desc * new urls * Updated naming of revit elements to include family type (#193)