oceanprotocol / df-web

https://df.oceandao.org/
0 stars 4 forks source link

3 boxes in "Rewards" section are useless, recommend deleting them. Also delete "Reward Programs" mid-title #633

Closed trentmc closed 10 months ago

trentmc commented 1 year ago

Background / motivation

If you go to df.oceandao.org then it will auto-redirect to df.oceandao.org/rewards.

Screenshot: rewards

Information architecture of the page:

  1. Key top-level info: round number, OCEAN available, avg APY, (your APY if connected), time to next rewards.
  2. Three "benefits" boxes: "veOCEAN: .. battle-tested contracts ..", "Weekly Rewards: .. run weekly ..", "For everyone: numerous participants can benefit"
  3. Detail info for each DF stream: Passive, Active: {Volume, Challenge}. For each stream, rewards available, (your rewards if connected), (claim rewards if connected)

Background / motivation 2

We now have pretty good DF docs, at https://docs.oceanprotocol.com/rewards

Screenshot: docs

These docs are linked directly from the "about" button.

Background / motivation 3

There's now a "Data Farming" hub page directly from oceanprotocol.com site, specifically at https://oceanprotocol.com/earn/data-farming. It got shipped recently, on approx Aug 14, 2023.

Screenshot: hub

FYI it isn't perfect yet: it's missing the "Challenge DF" substream and more. But @idiom-bytes is on top of that, via specific issues in "site" repo.

On the three "benefits" boxes

Summary: benefits boxes are redundant, with not-so-useful information that clutters up the "Rewards" page

On "Reward Programs" mid-title

Right below the "benefits" boxes and above the Detail info for each DF stream, there's a mid-title that says "Reward Programs"

Problem: It doesn't really add any useful info.

Recommended TODOs

KatunaNorbert commented 1 year ago

Oh, didn't know about this page inside the website.

The purpose of those boxes was: -to serve a presentation purpose for users who land on the page for the first time. Useful when we don't have a presentation page like the recent one from website. -give a feedback and guidance by showing more advanced futures base on the state of the user's connected wallet ( If already locked we don't show lock related information rather highlight what possibilities it has after that ).

Now that we have the website update we can iterate on this component

trentmc commented 1 year ago

Now that we have the website update we can iterate on this component

I think we can remove it. Especially if we have a one-sentence summary at the top.