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: add creationMode property to RqgActorData #639

Merged
merged 3 commits into from
Nov 21, 2023

Conversation

Moonpile
Copy link
Contributor

@Moonpile Moonpile commented Oct 8, 2023

No description provided.

@Moonpile Moonpile requested a review from wake42 October 8, 2023 02:05
@Moonpile
Copy link
Contributor Author

Moonpile commented Oct 8, 2023

Starting off really small!

@Moonpile
Copy link
Contributor Author

@wake42 So this was a very silly small PR. I changed the property name to "editMode" based on your comments.

@@ -14,6 +14,7 @@ export interface CharacterDataSourceData {
// --- Derived / Convenience Data Below ---
extendedName: string;
attributes: Attributes; // Most are derived // TODO Split / move data?
editMode: boolean;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just formatting, but both editMode and extendedName should be above the comment // --- Derived / Convenience Data Below --- since they are not derived

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@wake42 wake42 marked this pull request as ready for review November 21, 2023 19:42
@Moonpile Moonpile merged commit 43d0ed5 into main Nov 21, 2023
1 check passed
@Moonpile Moonpile deleted the Moonpile/Character-Creation-Mode-Property-#632 branch November 21, 2023 20:53
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.

None yet

2 participants