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 Thieves' Night On the Town -- Tnott #2824

Merged
merged 3 commits into from
Jun 21, 2023
Merged

Conversation

megapatato
Copy link
Contributor

Thieves Guild Requirements and Thieves' Night on the Town

Both plugins declare entries for MarkarthNepossHouse "Nepos's House" [CELL:00016DFB]

Thieves Guild Requirements adds a persistent reference. Thieves Night on the Town changes the cell's flags and locklist.

Both modifications play fine together, but loading TGR after Tnott will undo Tnott's changes; loading Tnott after TGR will preserve the changes from both.

If submission needs style cleaning / adjustment, please advise.

… the Town

Both plugins declare entries for MarkarthNepossHouse "Nepos's House" [CELL:00016DFB]

Thieves Guild Requirements adds a persistent reference.
Thieves Night on the Town changes the cell's flags and locklist.

Both modifications play fine together, but loading TGR after Tnott will undo Tnott's changes; loading Tnott after TGR will preserve the changes from both.
@megapatato megapatato changed the title Add sorting rule for Add sorting rule for TGR & TNOTT Jun 16, 2023
masterlist.yaml Outdated Show resolved Hide resolved
@sibir-ine sibir-ine changed the title Add sorting rule for TGR & TNOTT Add Thieves' Night On the Town -- Tnott Jun 17, 2023
* C.Locklist, as MarkarthNepossHouse cell changes from being a single NPC, to a formlist of residents
* C.MiscFlags, as MarkarthNepossHouse cell is not longer flagged as a public area
* C.Owner, to reset ownership of MorthalThaumaturgistHouse and prevent locking issues with the new NPC
* Keywords, as many Locations gained the keywords TGRWealthyHome and TGBusiness
* Names, as a couple Locations had their FULL - Name record changed to match the name of the cell they contain, instead of a non-existant one.
@megapatato
Copy link
Contributor Author

The style suggestion was accepted. Five BashTags were also added.

If there's additional adjustments, please advise.

masterlist.yaml Show resolved Hide resolved
masterlist.yaml Outdated Show resolved Hide resolved
- Remove C.MiscFlags as its intended change should already be covered by usage of C.Owner tag
- Remove extra newline at end of file
@megapatato
Copy link
Contributor Author

The suggestions are now included.

I tried accepting the suggestions directly on this PR thread, but after clicking "Commit suggestion", I would get some error about the diff being updated and requiring a refresh, but no commit would have been added. So I manually committed on my local repo.

Copy link
Member

@sibir-ine sibir-ine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, it happens sometimes, not sure why. Looks good though, thanks!

@sibir-ine sibir-ine merged commit 89b71e7 into loot:v0.18 Jun 21, 2023
1 check passed
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