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

Added VassalGameFileName property #12763

Merged

Conversation

riverwanderer
Copy link
Collaborator

Creates a property VassalGameFileName (akin to existing read-only properties such as ModuleVersion).
It is set to the file name of the most recent VSAV/VLOG.

Also fixes two issues with managing window title names from the Wizard which would have impeded the PR.

@riverwanderer riverwanderer added bug Something isn't working enhancement New feature or request labels Sep 29, 2023
@riverwanderer riverwanderer added this to the 3.7.2 milestone Sep 29, 2023
@riverwanderer riverwanderer self-assigned this Sep 29, 2023
@riverwanderer riverwanderer added the Ready for Review Ready to be reviewed for Merging label Sep 29, 2023
@BrentEaston BrentEaston modified the milestones: 3.7.2, 3.8.0 Oct 2, 2023
@BrentEaston
Copy link
Collaborator

This really needs to be a 3.8.0 enhancement. We can't have functionality that a module may use that is present in some versions of 3.7, but not others.

@riverwanderer
Copy link
Collaborator Author

This really needs to be a 3.8.0 enhancement. We can't have functionality that a module may use that is present in some versions of 3.7, but not others.

Thanks for the feedback. Shall I split out the bug fix element from the new functionality or just leave it all until v3.8.0 ?

@BrentEaston
Copy link
Collaborator

Split out the additional functionality. Good to get any bug fixes in asap, even small.

# Conflicts:
#	vassal-app/src/main/java/VASSAL/build/GameModule.java
@riverwanderer riverwanderer removed the bug Something isn't working label Nov 24, 2023
@uckelman uckelman changed the title VassalGameFileName property. Includes fixes for handling file name fr… Added VassalGameFileName property Jan 17, 2024
@uckelman uckelman merged commit 0867270 into vassalengine:master Jan 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Ready for Review Ready to be reviewed for Merging
Projects
None yet
3 participants