Skip to content
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

Merged
merged 10 commits into from
Feb 28, 2024

Conversation

JR-Morgan
Copy link
Member

@JR-Morgan JR-Morgan commented Feb 27, 2024

  • 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
@JR-Morgan JR-Morgan marked this pull request as draft February 27, 2024 15:34
@JR-Morgan JR-Morgan marked this pull request as ready for review February 27, 2024 19:54
@JR-Morgan JR-Morgan changed the title Cnx 8778 update connectors to use new fe2 terminology feat(UI): CNX 8778 update connectors to use new fe2 terminology Feb 28, 2024
@JR-Morgan JR-Morgan changed the title feat(UI): CNX 8778 update connectors to use new fe2 terminology feat(UI): [CNX 8778] update connectors to use new fe2 terminology Feb 28, 2024
@JR-Morgan JR-Morgan changed the title feat(UI): [CNX 8778] update connectors to use new fe2 terminology feat(UI): CNX-8778 update connectors to use new fe2 terminology Feb 28, 2024
@JR-Morgan JR-Morgan changed the title feat(UI): CNX-8778 update connectors to use new fe2 terminology feat(UI): CNX-9070 update connectors to use new fe2 terminology Feb 28, 2024
@@ -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.
Copy link
Member

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?

Copy link
Member Author

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.

Copy link
Member

@clairekuang clairekuang left a 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!

@JR-Morgan JR-Morgan added this to the 2.19 milestone Feb 28, 2024
@JR-Morgan JR-Morgan merged commit 24e7f02 into dev Feb 28, 2024
8 checks passed
@JR-Morgan JR-Morgan deleted the CNX-8778-Update-Connectors-to-use-new-FE2-Terminology branch February 28, 2024 18:53
JR-Morgan added a commit that referenced this pull request May 14, 2024
* 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)
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