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

chore: make scripted activity derived from activity #1363

Merged
merged 4 commits into from
Dec 29, 2023

Conversation

aronwk-aaron
Copy link
Member

@aronwk-aaron aronwk-aaron commented Dec 27, 2023

this paves the way for other component being properly derived from the activity component
Some docs on this on this branch

Tested that scripted activities such as foot races and the combat challenge still work as expected

this paves the way for other component being properly derived from the activity component
@aronwk-aaron aronwk-aaron force-pushed the chore--make-scripted-activity-drived-from-activity branch from da3bd63 to 913b844 Compare December 28, 2023 00:12
@aronwk-aaron aronwk-aaron marked this pull request as ready for review December 28, 2023 00:13
Jettford
Jettford previously approved these changes Dec 28, 2023
Jettford
Jettford previously approved these changes Dec 28, 2023
Copy link
Collaborator

@EmosewaMC EmosewaMC left a comment

Choose a reason for hiding this comment

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

feedback given in call

Copy link
Collaborator

@EmosewaMC EmosewaMC left a comment

Choose a reason for hiding this comment

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

I assume this has been tested in various activities like Battle of Nimbus Station, Avant Gardens Survival and Combat Challenge?

@aronwk-aaron aronwk-aaron merged commit fddf999 into main Dec 29, 2023
3 checks passed
@aronwk-aaron aronwk-aaron deleted the chore--make-scripted-activity-drived-from-activity branch December 29, 2023 01:11
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