Betonquest: ConversationIO option in conversation.yml

Created on 25 Apr 2017  路  7Comments  路  Source: BetonQuest/BetonQuest

Currently you specify your ConfigurationIO in the Plugins config.yml and can only have one IO.

I suggest adding the posibility to specify the IO for each conversation independently in the conversation.yml like in the following example:

quester: hans
conversationIO: chest
stop: false
first: ...
NPC_options:
...
player_options:
...

If there is no IO specified i'd just use the one specified in the config.yml.

This would be a very Powerfull feature because some interesting hidden features of the ChestIO (like the Quest GUI) would also be possible for Servers which decided to use for example the TellrawIO.

Most helpful comment

Yes, I'd love the PR 馃槃

All 7 comments

I now had a look in your code and saw a way to add this.
I could try to implement it on my own and will then create a pull request.

Why do you want to do that? I'd consider a consistent user experience pretty important...

Like I already mentioned: The ChestIO and the new IO comming with #424 have a lot of hidden usages, less for creating conversations, more for creating BQ-dependent GUIs.
If you don't know about those I recommend reading the Quest GUI section in Tips and Tricks.

I'd use this feature for this, not for creating simple conversations. For that I (and also the huge majority of players on our server) like the tellrawIO much more.
So this would have no impact on the consistency of the user experience (which is indeed pretty important, I totaly agree with you on that point)

And for people who don't want to use this nothing would change.
I hope I could clarify to you what is my intention. :)

Okay, good point. Of course I know about the custom GUI-"hack", but since I haven't used it so far I actually didn't think of the problem of that all conversations would need to be through the chest GUI...

If you have the time to code something, I think a PR will always be appreciated ^^

Yes, I'd love the PR 馃槃

Just created the pull request. I hope it helps! #535

Merged, thanks ^^

Was this page helpful?
0 / 5 - 0 ratings

Related issues

joblo2213 picture joblo2213  路  3Comments

NotJustPizza picture NotJustPizza  路  3Comments

soliddanii picture soliddanii  路  5Comments

Rouchesta picture Rouchesta  路  5Comments

Tiplish picture Tiplish  路  5Comments