search

PitchforkAssistant / devvit-flair-assistant

Attempt at replacing Flair Helper with a Reddit Developer Platform based project.
BSD 3-Clause "New" or "Revised" License
4 stars 1 forks source link

Add ability to change post flair #2

Closed PitchforkAssistant closed 6 months ago

PitchforkAssistant commented 10 months ago

Likely identical to the way userFlair is defined, just the same for postFlair. Config generator will also need to be updated.

There's also the question of what to do with the existing clearPostFlair property. Does it get deprecated? If not, which one takes priority?

epicmindwarp commented 7 months ago

clearPostFlair is fine to stay, because some people will still want it to clear.

PitchforkAssistant commented 7 months ago

clearPostFlair is fine to stay, because some people will still want it to clear.

Should I also add a clearUserFlair toggle? That might be the most elegant solution, since that would make the PostFlair and UserFlair configs match up.

epicmindwarp commented 7 months ago

Yes, a toggle makes sense.

I would put clear flair as last priority.

PitchforkAssistant commented 6 months ago

Implemented as of commit c2b2a227cd1f068253b35ce7b82509c426e83121