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

[Manager] Add application name to task properties window #5827

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

Vulpine05
Copy link
Contributor

Provides the general application name. This allows the user to easily identify the application that will run the task. Helpful for finding which application to reference in app_config.xml.

Fixes #5166

Description of the Change
Under the properties window, a wxString is added that displays the application name.

image

Release Notes
[Manager] Added application name to task's properties window.

Provides the general application name.  This allows the user to easily identify the application that will run the task.  Helpful for finding which application to reference in app_config.xml.
@AenBleidd
Copy link
Member

@Vulpine05, are you working on this as a Hacktoberfest participant?

@Vulpine05
Copy link
Contributor Author

@AenBleidd, yes I am.

@AenBleidd AenBleidd merged commit 430daae into BOINC:master Oct 1, 2024
146 checks passed
@Vulpine05 Vulpine05 deleted the Vulpine05_5166 branch October 1, 2024 22:24
@RichardHaselgrove
Copy link
Contributor

@Vulpine05 - Could you consider adding the plan_class to the display as well, please? As @davidpanderson says in #5830, the plan class is needed as well to completely define an application version, and as the user manual says, it is also specified for use in an app_config.xml file. (NB - plan_class is optional, and may be blank)

That would complete a useful addition - thank you for the work so far.

@Vulpine05
Copy link
Contributor Author

@RichardHaselgrove , yes, I definitely can do that. Good point!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Display internal application name in task properties.
3 participants