Revamped Github Section of Landing Page

seanK04 commented 2 months ago

Description

Currently, clicking 'GitHub' on the navbar of the landing page leads to the main app github repository. I implemented a whole page containing info and links to the 3 PearAI repositories - the app repo, submodule repo, and landing page repo. I've also added a bit of additional information such as relevant contribution links and an explanation of and links related to the 3 quest boards.

Related Issue

I made this page in order to address "Revamping Github section of Landing Page https://github.com/trypear/pear-landing-page/issues/169".

Screenshots

Checklist

[X] I have tagged the issue in this PR.
[X] I have attached necessary screenshots.
[X] I have provided a short description of the PR.
[X] I ran yarn build and build is successful
[X] My code follows the style guidelines of this project.
[X] I have added necessary documentation (if applicable)

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
pear-landing-page	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 24, 2024 4:59pm
pear-landing-page-main	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 24, 2024 4:59pm

vercel[bot] commented 2 months ago

@seanK04 is attempting to deploy a commit to the PearAI Team on Vercel.

A member of the Team first needs to authorize it.

Fryingpannn commented 2 months ago

Ty! Here are my comments

I think the github link should still link directly to the github repo of pearai app
instead this new page u have, should be a separate page entirely for contributors, and reached by clicking the Contributing 101 button on bottom of landing page (would only need one button there instead of two as there is now)
We need to put more thought into the structure of this page, rn there's everything thrown at the user it is hard to know where to look or to click. Even the "Want ot Contribute" section, there's too many links, contributing 101 and contributing guide is redundant. Maybe need to modify the docs. Also can remove "contributor roles". Need to think about the information here, and condense. Currently bad UX since too much info scattered.
Remove the buttons, they are already hyperlinks in the text for the same link.
Quest boards are a subsection of "Want to Contribute?", should not be another section. Also no need to put actual names there since they will change, will be hassle to keep this updated. Instead should explain that there are some general quests (they have [General] in title), and then the others that are temporary. I can prob provide improvements to the description in due time, but u can structure the positioning first.

rohankokkulabito commented 1 month ago

Heya @nang-dev & @Fryingpannn , Sorry to jump into your conversation, but I’ve got a little favor to ask! 😄

I’m Rohan, the Developer Community Manager at Bito. I’d like to run our AI Code Review Agent on your PRs. Think of it as letting AI take the “first pass” on your code reviews—covering security, performance, scalability, and optimization.

You don’t need to do anything for this initial setup. I’ll handle running the Agent and will share any potential fixes and suggestions it comes up with.

Later on, if you’re up for it, you can try it on your other PRs as well. I couldn’t help but notice one with 646 file changes 👀—definitely a good candidate for a thorough review!

Looking forward to seeing what the Agent can uncover and hoping it helps make your code even better.

rohankokkulabito commented 1 month ago

Code Review Agent Run #27725c

AI Based Review: ✔️ Successful

High-level Feedback

Revert the change in the 'yarn.lock' file to ensure all required version ranges for 'tailwindcss' are included, maintaining compatibility with other packages. Consider validating and sanitizing inputs for new components to prevent security risks. Ensure dependencies are up-to-date and manage unsupported versions gracefully to avoid process failures.

Actionable Issues

:page_facing_up: yarn.lock

Issues: Total - 1, High importance - 1

Line 4773-4773

🔴 High importance - 1

AI Code Review powered by

rohankokkulabito commented 1 month ago

I have ran the agent on one of your other PRs. Here's the suggestion it provided.

Can't wait to showcase you the potential benefit of using Bito as part of your PR reviewing workflow.

Let me know what you think.

Fryingpannn commented 4 weeks ago

Closing since we have docs page now <3

trypear / pear-landing-page