Skip to content

Mini Houses Added in AOS#309

Merged
Xoduz merged 8 commits intoUOX3DevTeam:developfrom
DragonSlayer62:minihouses-add-ons
Jan 28, 2025
Merged

Mini Houses Added in AOS#309
Xoduz merged 8 commits intoUOX3DevTeam:developfrom
DragonSlayer62:minihouses-add-ons

Conversation

@DragonSlayer62
Copy link
Contributor

All the Mini House Add-Ons

Copy link
Contributor

@strattond strattond left a comment

Choose a reason for hiding this comment

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

Minor suggestion, but looks reasonable to me. Haven't done a full box and dice cross-validation of content.

@Xoduz Xoduz added Changes Requested Change requested as part of Review and removed Ready for Review labels Jan 24, 2025
@DragonSlayer62 DragonSlayer62 added Ready for Review and removed Changes Requested Change requested as part of Review labels Jan 25, 2025
@Xoduz Xoduz added Changes Requested Change requested as part of Review and removed Ready for Review labels Jan 26, 2025
added to addmenu
added origin tag to the deeds.
@DragonSlayer62 DragonSlayer62 added Ready for Review and removed Changes Requested Change requested as part of Review labels Jan 27, 2025
@Xoduz Xoduz self-requested a review January 28, 2025 08:17
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add origin=aos to the deeds, and add a new section for them (ITEMMENU 261?) in the Deeds section (ITEMMENU 4) of the 'add menu, so they can be found.

@Xoduz Xoduz dismissed strattond’s stale review January 28, 2025 08:18

Requested change (to use GET) is not currently supported and would require a significant code change to support, beyond the scope of this PR :)

@Xoduz Xoduz merged commit fd4b542 into UOX3DevTeam:develop Jan 28, 2025
@DragonSlayer62 DragonSlayer62 deleted the minihouses-add-ons branch April 27, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants