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

[minor] Explicitly Load Plugin Dependencies to Avoid Resaving the Project #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

varshini-senthil
Copy link
Contributor

What does this Pull Request accomplish?

This Pull Request updates the Create Plugin.vi to explicitly load the dependencies of the plugin library VIs before saving the project, ensuring that the dependencies of the VIs are loaded in the project.

Context

  • Currently, when creating a new plugin using the Generator tool, the Create Plugin.vi adds the plugin library to the active project programmatically. However, this action does not automatically load the dependencies of the plugin in the project window. As a result, when we close and reopen the project, an * symbol appears in the project window, indicating unsaved changes, which actually point to the addition of the plugin's dependencies to the project window.
  • To address this, I have added logic to load the plugin VIs (i.e., the top-level plugin VI) into memory before saving the project. This ensures that the dependencies are explicitly loaded in the project window before saving the project, avoiding the above-mentioned scenario.

Why should this Pull Request be merged?

This Pull Request should be merged to ensure that the dependencies of the plugin VIs are loaded into the project before saving, thereby preventing unsaved changes related to the plugin's dependencies.

What testing has been done?

Manual testing has been conducted by creating a plugin and ensuring that the above-mentioned scenario is not encountered.

@varshini-senthil
Copy link
Contributor Author

  • Dependencies section when a new plugin is created in the project, using the Generator tool.

image

  • Dependencies section on reopening the project window.

image

@varshini-senthil
Copy link
Contributor Author

Create Plugin.vi
Code diff
Before:
image

After:
image

@varshini-senthil varshini-senthil marked this pull request as ready for review March 14, 2025 05:38
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