Skip to content

Conversation

@JanRogge
Copy link

@JanRogge JanRogge commented Feb 4, 2026

2 Weeks ago i let AI fix a lot of secret errors to have tiptac working for the time.

These fix are in no way correct or should be merged but maybe it helps you getting an idea of what problems exist with secrets.

Most of the tests i did where with /run SetCVar("secretChallengeModeRestrictionsForced", 1) at the Dornogal test dummies this is to test the restrictions active in a Key.

I also had these cvars active.

secretCombatRestrictionsForced
secretEncounterRestrictionsForced
secretMapRestrictionsForced

Looking forward to your offical version.

@frozn
Copy link
Owner

frozn commented Feb 4, 2026

Thanks for the PR! 👍

It's really crazy (and helpful) what AI is capable about. 🤯

@frozn frozn added 1_bug Something isn't working 2_to develop Development can begin labels Feb 4, 2026
@frozn
Copy link
Owner

frozn commented Feb 4, 2026

I didn't know the secret restriction CVARs. Very helpful!

Hint: There's a secretPvPMatchRestrictionsForced too.

@JanRogge
Copy link
Author

JanRogge commented Feb 6, 2026

Glad it could help you a bit if you need anything else hit me up.

@burntorangejedi
Copy link

@JanRogge - This is great - I just gave it a go and it seems to be working ok. I can't tell if there is anything missing, but so far nothing seems broken.

@frozn - can we merge this into the main branch?

@JanRogge
Copy link
Author

JanRogge commented Feb 9, 2026

@burntorangejedi This is by far not production ready and also would require a lot of changes to be compatible with the changes frozn already did to make everything really compatible.

Feel free to use my fork till frozn is ready but i bet his work will 100% better then what this is ^^

@burntorangejedi
Copy link

@JanRogge & @frozn - totally up to y'all. i'm only saying it's working fine for me, at least, but ymmv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1_bug Something isn't working 2_to develop Development can begin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants