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 (lfgsolo): clean up #120

Merged

Conversation

acidmanifesto
Copy link
Contributor

Changes proposed:

  • Reduce script diff from master. LFG Solo was literally over 90% core edit with a on/off bool script in a custom script folder. It made no sense. Time to revisit the idea of a cleanup.
  • Remove Lfg_Solo.cpp from custom script folder
  • Remove announcement on login and in conf, NO BODY CARES about the chatbox spam on login and to have a OnLogin constantly hit for something as frivleous as that is kinda insane.
  • Move SoloLFG.Enable bool to the exsisting LFGPlayerScript::OnLogin.

Target branch(es): 3.3.5-lfgsolo

Issues addressed: Fixes #

Tests performed: (Does it build, tested in-game, etc)

Known issues and TODO list:

  • [ ]
  • [ ]

Reduce script diff from master. LFG Solo was literally 90% core edit with a on off bool in a custom script folder. It made no sense. Time to revisit the idea of a cleanup
Remove Lgf_Solo.cpp from custom script folder
Remove announcement on login and in conf
Move  SoloLFG.Enable bool to the exsisting LFGPlayerScript::OnLogin.
@acidmanifesto acidmanifesto changed the title chore (lfg-solo): clean up chore (lfgsolo): clean up Oct 20, 2023
@jackpoz jackpoz merged commit 99e072f into TrinityCore:3.3.5-lfgsolo Oct 23, 2023
2 checks passed
@acidmanifesto acidmanifesto deleted the lfg-solo-cleanup-rewrite branch October 23, 2023 20:35
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