Fix punycode handling to be less stupid

Separated screen_name_ui to use for things where it's rendered, and just keep using unmodified screen_name in logic. I don't know if this is the way it should work? or should it use internally the decoded one just without the emoji?

Closes: #1064 (closed)

changed the description

added 1 commit

• 49aa10e1 - add screen_name_ui to tests

Compare with previous version

it might be better to keep using decoded unicode in screen_name, and add the world emoji only in a few places where we display the name, using simple functions. I don't know if mentioning works like that though

EDIT: scratch that, thank god for previews for quick testing, the chosen approach in this MR makes sense and according to @lanodan it seems to work

approved this merge request

added 4 commits

• 49aa10e1...a83fdbbd - 3 commits from branch develop
• 237f272d - Merge branch 'develop' into 'fix/punycode-buggy'

Compare with previous version

enabled an automatic merge when the pipeline for 237f272d succeeds

mentioned in commit aa5cb3d1

merged