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

Inconsistent Priority Behaviour #220

Open
AssiduousTentativeFrog opened this issue Dec 3, 2024 · 3 comments
Open

Inconsistent Priority Behaviour #220

AssiduousTentativeFrog opened this issue Dec 3, 2024 · 3 comments

Comments

@AssiduousTentativeFrog
Copy link

Hi, I have encountered an issue with the priority feature when using scripts:

When setting up a priority list like follows
image
It will show that the list matches my current party, however when the script gets activated I get this message:
image
I believe the issue could be caused by the way the priority list loops over each priority and uses each item inside it to get a party member from the party directly therefore being able to get the same party member more than once, it could be fixed by making a copy of the party list at the time GetPlayers gets called so it can be passed down to IsInParty. This way you could remove the party members that already got select by another priority line therefore allowing us not to have to refill the UI every time we enter a new pf.

Have a lovely day

@Limiana
Copy link
Collaborator

Limiana commented Dec 3, 2024

Hello! Please don't select so many jobs. One priority slot should explicitly match exactly one player's job or name.

@Limiana
Copy link
Collaborator

Limiana commented Dec 3, 2024

Message will be corrected later to inform user about this error

@AssiduousTentativeFrog
Copy link
Author

That's a bit disappointing if I'm honest, as refilling every script that has a priority in every new party I join / every time someone leaves takes a bit longer for every new script.
Do you think it would be possible to support tags like <1> in those scripts so we can "passively" set up all those scripts at once by ordering our party list ?

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

2 participants