Skip to content

out-of-the-box support for Astaria's hashing and colorisation of the mapper - #194

Open
araskal wants to merge 7 commits into
IRE-Mudlet-Mapping:masterfrom
Astaria-Mud:master
Open

out-of-the-box support for Astaria's hashing and colorisation of the mapper#194
araskal wants to merge 7 commits into
IRE-Mudlet-Mapping:masterfrom
Astaria-Mud:master

Conversation

@araskal

@araskal araskal commented Jun 20, 2024

Copy link
Copy Markdown

After a fair bit of trial and error this is all working for Astaria - though I am not sure whether there's any issues with other MUD's, I tried to limit any changes.

NathanMArgenti and others added 5 commits May 13, 2023 16:44
added astaria to script - todo: convert room id to allow for hash
Add support for games that operate on hashes (IRE-Mudlet-Mapping#186)
Added support for Astaria's game env and colorisation

@vadi2 vadi2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR! A couple of comments below -

Comment thread mudlet-mapper.xml
end
raiseEvent("mmp clear externals")
mmp.gotoRoom(closestFeature, dashtype, "room")
mmp.gotoRoom(closestFeature, dashtype, "area")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What does this change do?

Comment thread mudlet-mapper.xml
if gmcp.Room.Info.num then
centerview(gmcp.Room.Info.num)
elseif gmcp.Room.Info.roomhash then
print(tostring(mmp.roomidFromHash(gmcp.Room.Info.roomhash)))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Debug?

@vadi2

vadi2 commented Aug 3, 2025

Copy link
Copy Markdown
Contributor

@araskal is this good to go?

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