Skip to content

Conversation

@cecell
Copy link
Contributor

@cecell cecell commented Dec 19, 2025

Quotation from Relationship Dialogue Overhaul - RDO SE page:

Load RDO AFTER the following mods listed below:

...

  • Alternate Start - Live Another Life (LAL)
    • A handful of records conflict between RDO and LAL. RDO includes the changes made to these conflicting records by LAL, and therefore should be loaded after LAL so that changes from both mods are applied.

A handful of records conflict between RDO and LAL. RDO includes the changes made to these conflicting records by LAL, and therefore should be loaded after LAL so that changes from both mods are applied."

https://www.nexusmods.com/skyrimspecialedition/mods/1187
'Sofia - RDO Patch.esp' is missing 'Relationship Dialogue Overhaul.esp' as a master and default sorting places it before RDO.
Comment on lines +15494 to +15496
after:
- '018Auri.esp'
- 'alternate start - live another life.esp'
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
after:
- '018Auri.esp'
- 'alternate start - live another life.esp'
after: [ '018Auri.esp' ]

This rule has come up frequently over the years (#576, #715, #1311) & was removed in #1312 since it doesn't work for every load order. Not much has changed since, so we should still leave it to user preference.

Comment on lines +15502 to +15506
- <<: *patch3rdParty
subs:
- 'Sofia - The Funny Fully Voiced Follower'
- '[Sofia Bug and Patch Hub](https://www.nexusmods.com/skyrimspecialedition/mods/70950)'
condition: 'active("SofiaFollower.esp") and not active("Sofia - RDO Patch.esp")'
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- <<: *patch3rdParty
subs:
- 'Sofia - The Funny Fully Voiced Follower'
- '[Sofia Bug and Patch Hub](https://www.nexusmods.com/skyrimspecialedition/mods/70950)'
condition: 'active("SofiaFollower.esp") and not active("Sofia - RDO Patch.esp")'

The patch message on RDO should be removed. One's enough, & it makes more sense on SofiaFollower.esp since it's from a patch hub for it.

Comment on lines +29907 to +29909
url: [ 'https://www.nexusmods.com/skyrimspecialedition/mods/70950' ]
# Missing RDO as a master
after: [ 'Relationship Dialogue Overhaul.esp' ]
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
url: [ 'https://www.nexusmods.com/skyrimspecialedition/mods/70950' ]
# Missing RDO as a master
after: [ 'Relationship Dialogue Overhaul.esp' ]
url:
- link: 'https://www.nexusmods.com/skyrimspecialedition/mods/70950/'
name: 'Sofia Bug and Patch Hub'
req: [ 'Relationship Dialogue Overhaul.esp' ]

Requirements are better for missing masters, they load the plugin after & display an error message if the required plugin is missing. Also, some tweaks to the URL for conformity & convenience.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh sweet. Thanks! I'll do better xD

@sibir-ine
Copy link
Member

Just FYI, you have to commit the suggestions, resolving the comments doesn't actually do that. You an either commit them from the comment (the Commit suggestion button) or locally.

@cecell
Copy link
Contributor Author

cecell commented Dec 20, 2025 via email

@cecell
Copy link
Contributor Author

cecell commented Dec 20, 2025

Ah, here's the problem. This is why I clicked resolve conversation:
Screenshot_20251220-094023

No matter how many times I click that it says that again and again.

Is there a git command that will force it?

@sibir-ine
Copy link
Member

Not that I know of, it's an issue with GitHub that pops up every once in a while. You'd have to manually make the changes and push to your branch. However, I'll just commit them since it's letting me do it.

@sibir-ine
Copy link
Member

Never mind, it's giving me the same error now. You'll have to manually make the changes unfortunately.

@cecell
Copy link
Contributor Author

cecell commented Dec 21, 2025 via email

@sibir-ine
Copy link
Member

It's a learning process, & we're happy to help anyone who's willing to learn. Did you push the changes? They'll show up in the PR once you've done so successfully. If you're not sure how, feel free to ask.

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