Document new rich presence url & status display type fields for Social SDK#7676
Conversation
foxfirecodes
changed the base branch from
main
to
lew/add-url-and-status-display-rich-presence-fields
colinloretz
added
the
not released
foxfirecodes
force-pushed
the
lew/add-url-and-status-display-rich-presence-fields-social-sdk
branch
from
5482bab to
3ea989f
Compare
| ``` | ||
| --- | ||
|
|
||
| ## Setting Field URLs |
There was a problem hiding this comment.
I think both of these sections would be better just below ## Setting Assets since it flows naturally from there.
I am wondering though if that section should be ## Setting Predefined Assets and this should be something like ## Setting Rich Presence Clickable Links or something like that - that describes what you are doing.
What do you think of that?
Down here we're in ## Rich Presence Without Authentication which feels like the wrong section.
There was a problem hiding this comment.
im good with moving them! i think that makes sense
i'm like 70% sure that you can set more than just predefined assets so i dont think renaming that section is necessary. as for renaming the field URLs section, i think it's redundant to say Rich Presence when we're on the rich presence page but i'm open to maybe doing "Setting Clickable Links" if that feels more intuitive to you than "Setting Field URLs"
so in the meantime ive moved the section, but left the headers the same. lmk your thoughts on if you still think the header could be reworded
markmandel
removed
the
not released
markmandel
left a comment
markmandel
left a comment
There was a problem hiding this comment.
Just did a preview - looks great. I agree with keeping the section titles, good call 👍🏻
markmandel
deleted the
lew/add-url-and-status-display-rich-presence-fields-social-sdk
branch
3 participants
Social SDK counterpart to #7674 -- separated out so we can wait until Social SDK has been released to update those docs