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

Add new preset: Dornogal Inn (Humming/Fidgeting) #44

Merged
merged 6 commits into from
Sep 29, 2024

Conversation

biggianteye
Copy link
Contributor

@biggianteye biggianteye commented Sep 15, 2024

This is the innkeeper in Dornogal. He is constantly making noises.

Mute the sounds heard in the inn in Dornogal. These are predominantly from innkeeper Ronesh, but also includes other Earthen fidgeting sounds that can be heard in the inn.

@funkydude
Copy link
Owner

Hey. I appreciate the pull request but when creating presets you need to be sure you are only adding the correct sounds.
You can do this easily in game by simply using the preview button in the addon:
image

Doing this you can see that at least the first 2 sounds sound like fighting sounds and are not related to him humming, so they should be removed from the list.

The last 2 sounds might also be questionable as they may be the sounds he makes when interacting with him, but maybe you intentionally want that?

@funkydude
Copy link
Owner

Do you plan on addressing this?

@biggianteye
Copy link
Contributor Author

Sorry for the delayed response.

Doing this you can see that at least the first 2 sounds sound like fighting sounds and are not related to him humming, so they should be removed from the list.

These do indeed sound like generic fighting sounds, but they are categorised as earthendwarfmale_fidget13, just like the humming is. They are audible in the inn, but not directly from Ronesh so I have removed them.

The last 2 sounds might also be questionable as they may be the sounds he makes when interacting with him, but maybe you intentionally want that?

These two are also audible in the inn but happenly infrequently, so I couldn't definitively attribute them to Ronesh, so I have removed them as well.

However these changes now make me wonder if this is a useful preset to have at all in its current state, given there are only three sounds left. If it were merged, I would still be manually muting the other four sounds. Alternatively, I could rename the preset to be about the inn rather than the NPC. What do you think?

Options, in no particular order:

  • Switch the preset to the inn.
  • It's useful as is. Merge it.
  • It's not useful. Scrap the PR.

I'm fine with any of those. Whichever fits in to what you want for the addon.

@funkydude
Copy link
Owner

funkydude commented Sep 26, 2024

Changing it to "Zone: Dornogal Inn (Humming/Fidgeting)" sounds good! Thanks for testing and verifying the sounds.

@biggianteye biggianteye changed the title Mute Innkeeper Ronesh Mute Dornogal Inn Sep 29, 2024
@biggianteye
Copy link
Contributor Author

Changing it to "Zone: Dornogal Inn (Humming/Fidgeting)" sounds good! Thanks for testing and verifying the sounds.

I've updated the PR to be about the inn rather than the NPC and reinstated the additional sounds you can hear in the inn.

@funkydude funkydude changed the title Mute Dornogal Inn Add new preset: Dornogal Inn (Humming/Fidgeting) Sep 29, 2024
@funkydude funkydude merged commit 57a3d0e into funkydude:master Sep 29, 2024
1 check passed
@funkydude
Copy link
Owner

Nice work, thanks!

@funkydude
Copy link
Owner

I think there is also a female version of this, just outside of the inn, if you're interested in adding that also.

@BloodMoonTalon
Copy link

Hey All, first of all ty for this addon
Ive just signed up to github just to request something
There are "Oathsworn Peacekeeper" right beside single target PVP Training Dummy in Dornogall
Ive tried every sound id inside wowhead to mute them but to no avail
If u could make a preset for them i would be so happy

@biggianteye biggianteye deleted the ronesh branch October 1, 2024 19:40
@biggianteye
Copy link
Contributor Author

I think there is also a female version of this, just outside of the inn, if you're interested in adding that also.

It's not something that's affected me directly, so I'll leave it as an itch for someone else to scratch.

@biggianteye
Copy link
Contributor Author

Nice work, thanks!

No worries. Thanks for merging it!

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.

3 participants