extension icon indicating copy to clipboard operation
extension copied to clipboard

feat(profile): famous contributors

Open gregoirelacoste opened this issue 3 years ago • 6 comments

closes #998

@MaartenLMEM you can choose any famous contributors you want by editing famousContributors.json in the order you want Be careful to write the name of the contributor exactly as in the backoffice Could you also check the translation ?

gregoirelacoste avatar Jun 11 '21 16:06 gregoirelacoste

The app configuration is done in env, we might want to change this but it should be discussed. See similar feature for popular contributors (this isn't used but should be on install popin) and preselected contributors.

POPULAR_CONTRIBUTORS_IDS=[53, 54, 59, 46, 65]
PRESELECTED_CONTRIBUTORS_IDS=[16, 24, 25, 27, 29, 31, 42, 43, 44, 53, 54, 55, 59, 60, 65, 66, 67, 74]

lutangar avatar Jun 13 '21 18:06 lutangar

The app configuration is done in env, we might want to change this but it should be discussed.

The goal on this feature is to create something simple to manage for @MaartenLMEM . So using ids are to much complex for @MaartenLMEM beacause it need to go to backoffice and note ids for each contributors. Name is much simplier because it's dispalyed in front, just copy/past.

Env var is a good option, but we also would like something administrable from github

gregoirelacoste avatar Jun 14 '21 07:06 gregoirelacoste

I tend to agree with @lutangar here, for the sake of consistency … @MaartenLMEM woud it be OK to store the list as an array of IDs that you would change here : https://github.com/dis-moi/extension/blob/master/.env.production ? 

@lutangar actually, this could even be extracted from this physical file to hosted ENV, to enable deploy less editing …

JalilArfaoui avatar Jun 14 '21 14:06 JalilArfaoui

@MaartenLMEM woud it be OK to store the list as an array of IDs that you would change here : https://github.com/dis-moi/extension/blob/master/.env.production ? 

All right for me, need already discussed with Grégoire and Johan.

MaartenLMEM avatar Jun 15 '21 06:06 MaartenLMEM

@gregoirelacoste where are we on this one ? Do you need any help ?

JalilArfaoui avatar Nov 09 '21 22:11 JalilArfaoui

@JalilArfaoui rebased, could you push ?

gregoirelacoste avatar Nov 17 '21 14:11 gregoirelacoste