https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5805

`user_settings_lib.js` - is this something you created? It's not in the repo.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5807

Yes, I forgot to add it. I just pushed a commit that includes it.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5810

I'm not really a fan of the way this was done.

I'm considering putting something in shell_lib.js that you can use instead.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5811

That's fine

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5812

With the latest changes in SBBS v3.20 (commit ae3253085e1), I don't think you any of this logic - file attachments don't use the user's configured download protocol, by default, so no need to prompt the user to set/change it.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/merge_requests/465#note_5821

I've removed this logic, and have also removed load/user_settings_lib.js

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net