Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Hide "Tools:" when the current page is not a PR/Repo/User #196

Closed
2 tasks done
diivi opened this issue Jun 25, 2023 · 4 comments · Fixed by #197
Closed
2 tasks done

Hide "Tools:" when the current page is not a PR/Repo/User #196

diivi opened this issue Jun 25, 2023 · 4 comments · Fixed by #197
Assignees
@punyamsingh
Labels
🐛 bug Something isn't working good first issue Good for newcomers released on @beta released
Milestone
Extension Features

Comments

@diivi
Copy link
Contributor

diivi commented Jun 25, 2023

Describe the bug

This is what I see when I am not on one of the pages described in the conditional statements here -

ai/src/popup/pages/home.tsx

Lines 145 to 146 in f2c55fa

{GitHubPageType === "REPO" && (
<button

image

Steps to reproduce

  1. Go to any page except the 3 pages above.
  2. Open the extension popup.
  3. Empty tools.

Browsers

Chrome

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@diivi diivi added the 🐛 bug Something isn't working label Jun 25, 2023
@diivi
Copy link
Contributor Author

diivi commented Jun 25, 2023

Easy fix by hiding the "Tools:" text in an else statement.

@diivi diivi added the good first issue Good for newcomers label Jun 25, 2023
@punyamsingh
Copy link
Contributor

.take

@punyamsingh punyamsingh mentioned this issue Jun 25, 2023
Merged
19 tasks
@bdougie bdougie added this to the Extension Features milestone Jun 27, 2023
@bdougie bdougie moved this from Todo to Done in Extension Status Board Jun 27, 2023
@bdougie bdougie moved this from Done to In Progress in Extension Status Board Jun 27, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Extension Status Board Jun 29, 2023
@github-actions
Copy link

🎉 This issue has been resolved in version 1.9.0-beta.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This issue has been resolved in version 1.9.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot mentioned this issue Jul 12, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@punyamsingh punyamsingh

Labels
🐛 bug Something isn't working good first issue Good for newcomers released on @beta released
Projects
Extension Status Board
Status: Done
Milestone
Extension Features
Development

Successfully merging a pull request may close this issue.

fix: Made the "Tools:" conditional according to the current page punyamsingh/open-sauced-ai
3 participants
@bdougie @diivi @punyamsingh