Skip to content

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
@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