naming changes & reorganization of buttons for deverification in adminJS

[divine-comedian]

We should consider re-naming some to make it more clear for admins

rename Verify and Reject on Project tab to "approve vouched" & "remove vouched"

change name of Revoke Badge = "Revoke GIVbacks Eligible"

anything else to add here? @NikolaCreatrix @WhyldWanderer

[WhyldWanderer]

I think that covers it Mitch!

[maryjaf]

• when I tap on "reject" button in project page the verified status is changed to NO(pic 1) but I still see "vouched" tag in project page (pic 2)

Is it a bug or the development hasn't been completed yet?

https://staging.giveth.io/project/testdraft?tab=givpower

@RamRamez @divine-comedian

[maryjaf]

• should the name of "verification status" be changed?

@divine-comedian

[RamRamez]

• when I tap on "reject" button in project page the verified status is changed to NO(pic 1) but I still see "vouched" tag in project page (pic 2)

Is it a bug or the development hasn't been completed yet?

https://staging.giveth.io/project/testdraft?tab=givpower

@RamRamez @divine-comedian

I'm not sure why we use both "verified" and "isGivbackEligible" fields for showing "vouched" badge in FE.
https://github.com/Giveth/giveth-dapps-v2/blob/2bcbfe0f90530e006f29467b8a701145d3c73ec3/src/components/views/project/ProjectBadges.tsx#L28

[divine-comedian]

@maryjaf these are good observations and we can follow them up in a later issue after this pending release - for now let's stick to the bullet points Ramin shared above

[divine-comedian]

tested this out - renaming looks good and the buttons functions still work as expected!