Skip to content

refactor: prb feedback #254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Feb 6, 2025
Merged

refactor: prb feedback #254

merged 5 commits into from
Feb 6, 2025

Conversation

PaulRBerg
Copy link
Member

My feedback on #238, and then some.

cc @smol-ninja @razgraf not sure if you wanna review this or we can just merge directly

Closes #252 and #251

Copy link
Member

@smol-ninja smol-ninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good (I had a very brief look), except two comments below:

Copy link
Member

@smol-ninja smol-ninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PaulRBerg
Copy link
Member Author

Thank you @smol-ninja, will wait for @razgraf to confirm if any feedback/concerns/queries

@razgraf
Copy link
Member

razgraf commented Feb 4, 2025

Looks good to me! The airdrops dashboard itself suffered some changes as well so if you want to

  1. change the screenshot with an updated one (new design for the airdrop cards)
  2. mention the activity section (to see all claims etc.)

it would be a good addition. It's not urgent through, we can always update later if you want to get this moving, will leave it up to you. Thanks for updating the other screenshots and sections!

refactor: dedup and simplify release history
refactor: delete promo video section
refactor: delete superfluous sections in CSV guide
refactor: get rid of "group" terminology
refactor: rename images
refactor: improve CSV guide
refactor: improve wording and order alphabetically
refactor: update batch creation limits
refactor: update Airdrops app guide
@PaulRBerg
Copy link
Member Author

Thanks, @razgraf.

I've updated the airdrop screenshots.

And created an issue for the activity section: #263

@PaulRBerg PaulRBerg merged commit 85ea9d9 into main Feb 6, 2025
1 check passed
@PaulRBerg PaulRBerg deleted the refactor/prb-feedback branch February 6, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simplify the CSV support guide by removing superfluous and difficult-to-maintain sections
3 participants