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

[Feature Request] Cross-Faction Dungeon Finder [$30] #2212

Closed
andoys opened this issue Aug 15, 2019 · 16 comments
Closed

[Feature Request] Cross-Faction Dungeon Finder [$30] #2212

andoys opened this issue Aug 15, 2019 · 16 comments

Comments

@andoys
Copy link

andoys commented Aug 15, 2019

Feature Request: Cross-Faction Dungeon Finder
Must work with CFBG Module Enabled

  • Can be Module
  • Core Edit

Configurable by Worldserver.conf to enable/disable


Did you fix this issue? Go claim the $30 bounty on Bountysource.

@andoys andoys changed the title Feature Request: Cross-Faction Dungeon Finder Feature Request: Cross-Faction Dungeon Finder [$15] Aug 15, 2019
@andoys andoys changed the title Feature Request: Cross-Faction Dungeon Finder [$15] Feature Request: Cross-Faction Dungeon Finder [$30] Aug 15, 2019
@andoys andoys changed the title Feature Request: Cross-Faction Dungeon Finder [$30] [Feature Request] Cross-Faction Dungeon Finder [$30] Aug 15, 2019
@Yehonal Yehonal changed the title [Feature Request] Cross-Faction Dungeon Finder [$30] [Feature Request] Cross-Faction Dungeon Finder Aug 15, 2019
@Yehonal Yehonal changed the title [Feature Request] Cross-Faction Dungeon Finder [Feature Request] Cross-Faction Dungeon Finder [$30] Aug 15, 2019
@Yehonal Yehonal added the Bounty label Aug 15, 2019
@Ali-Khazaee
Copy link
Contributor

I've find a way to implement this feature in ac without hardcore edits, After testing it i'll make a PR

@FrancescoBorzi
Copy link
Contributor

This feature should be an addition to the CFBG

@Ali-Khazaee
Copy link
Contributor

https://github.com/Ali-Khazaee/azerothcore-wotlk/tree/CFD

@FrancescoBorzi
Copy link
Contributor

@Ali-Khazaee can you make it as an addition to the CFBG module?

we should avoid patching the core directly

@Winfidonarleyan
Copy link
Member

I never understood why need to do cfbg and cfd together.
It's not the same thing, it's 2 different systems (bgs and dungeons) it really annoys me when say it's the same

@Winfidonarleyan
Copy link
Member

Winfidonarleyan commented Aug 20, 2019

@Ali-Khazaee, please move all code from the core to module

@Ali-Khazaee
Copy link
Contributor

@FrancescoBorzi Alright, I'll make it as a separate module

@Ali-Khazaee
Copy link
Contributor

I've just realized making this as module, Requires to add new hooks to the core

@FrancescoBorzi
Copy link
Contributor

@Ali-Khazaee that's fine, you can add the hooks to the core when needed

@BarbzYHOOL
Copy link
Member

BarbzYHOOL commented Aug 20, 2019

A module for crossfaction dungeon finder, a module for crossfaction battlegrounds, a module for crossfaction ICC, what else??

When we enable crossfaction, we want all these things in ONE. All in one. Imo it should be in the core or all in one big module. In the core there is already crossfaction chat, grouping etc...........................

@FrancescoBorzi
Copy link
Contributor

A module for crossfaction dungeon finder, a module for crossfaction battlegrounds, a module for crossfaction ICC, what else??

When we enable crossfaction, we want all these things in ONE. All in one. Imo it should be in the core or all in one big module. In the core there is already crossfaction chat, grouping etc...........................

you got a point, what I don't want is to break/complicate/alter the current behaviour when introducing such extra features

@Ali-Khazaee @BarbzYHOOL I don't mind if it's in the core, as long as when:

AllowTwoSide.DungeonFinder = 0

it has NO side effects at all

@BarbzYHOOL
Copy link
Member

yes, it must be properly coded, if the author (most likely ali-khazaee) can't, then well a module will do the trick but in the end it should all be in the core OR we take all the crossfaction system OUT of the core and put all into a big module

@Ali-Khazaee
Copy link
Contributor

@Ali-Khazaee
Copy link
Contributor

I've tried to keep this feature simple and not too much hardcore code

@Malow
Copy link

Malow commented Oct 14, 2019

Isn't cross-faction dungeon finder already working by default in AC? At least on my server with the cross-faction battle ground module installed and all cross-faction flags enabled in the config I can queue with 2 Hordes and 3 Alliance to Random Dungeon and they all get matched together as a group.

@Helias
Copy link
Member

Helias commented Aug 23, 2020

@Malow, so can we close this?

Is there a way to move the bounty to another issue?

@Yehonal Yehonal added the Bounty label Dec 1, 2020
@FrancescoBorzi FrancescoBorzi changed the title [Feature Request] Cross-Faction Dungeon Finder [$30] [Feature Request] Cross-Faction Dungeon Finder Dec 1, 2020
@Yehonal Yehonal changed the title [Feature Request] Cross-Faction Dungeon Finder [Feature Request] Cross-Faction Dungeon Finder [$30] Dec 7, 2020
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

No branches or pull requests

8 participants