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

Updated /fmk, /trekduel, /tuvix, and /dustbuster to include l… #587

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rahmcoff
Copy link
Collaborator

@rahmcoff rahmcoff commented Feb 2, 2025

…inks to Memory Alpha

Also, updated the list of characters a bit

There are issues with a lot of the links, but I’m not sure the best way to handle it. It would require hard coding some of the links somewhere.

…inks to Memory Alpha

Also, updated the list of characters a bit
@zmattingly
Copy link
Collaborator

Hm, do you have some examples offhand where the links aren't working? Looks like MA has a way of getting people to the right place eventually if they do punch in a name wrong/etc? https://memory-alpha.fandom.com/wiki/MA_Help:Redirect

@zmattingly
Copy link
Collaborator

zmattingly commented Feb 2, 2025

Oh wait, duh I see, like Changeleader and etc. 🤔

Not sure of an elegant way to get around that either. Yeah maybe just hardcoding them in with a basic JSON file that has optional fall-through links for stuff like that.

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.

2 participants