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.

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

feat: view repos on opensauced #218

Merged
merged 2 commits into from
Jul 18, 2023
Merged

feat: view repos on opensauced #218

merged 2 commits into from
Jul 18, 2023

Conversation

diivi
Copy link
Contributor

@diivi diivi commented Jul 17, 2023

Description

Add a button to the popup which redirects the user to the repo's page on insights.opensauced.
Given that it exists on opensauced.

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

#14

Mobile & Desktop Screenshots/Recordings

Peek 2023-07-17 16-41

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 docs.opensauced.pizza
  • 🍕 dev.to/opensauced
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link
Member

@Anush008 Anush008 left a comment

Choose a reason for hiding this comment

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

Works @diivi.
Can you update the state variable name and the prURL key to pageURL as these aren't restricted to PR pages anymore?

export const usGetGitHubPageInfo = () => {
const [GithubPRPage, setGithubPRPage] = useState<GitHubPageInfo>({ prUrl: "", prTitle: "", type: "unknown" });

Currently we are renaming while destructuring here.

const { prUrl: pageURL, prTitle, type: GitHubPageType } = usGetGitHubPageInfo();

@bdougie bdougie merged commit f33d9fd into beta Jul 18, 2023
github-actions bot pushed a commit that referenced this pull request Jul 18, 2023
## [1.12.0-beta.1](v1.11.1...v1.12.0-beta.1) (2023-07-18)

### 🍕 Features

* view repos on opensauced ([#218](#218)) ([f33d9fd](f33d9fd))
@github-actions
Copy link

🎉 This PR is included in version 1.12.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Anush008 Anush008 deleted the feat/view-repos-on-opensauced branch July 18, 2023 14:25
zer0and1 pushed a commit to zer0and1/open-sauced.ai that referenced this pull request Jul 26, 2023
## [1.12.0-beta.1](open-sauced/ai@v1.11.1...v1.12.0-beta.1) (2023-07-18)

### 🍕 Features

* view repos on opensauced ([#218](open-sauced/ai#218)) ([f33d9fd](open-sauced/ai@f33d9fd))
@bdougie bdougie mentioned this pull request Jul 31, 2023
bdougie added a commit that referenced this pull request Jul 31, 2023
* feat: view repos on opensauced (#218)

* feat: view repos on opensauced

* change var names

* chore(minor): release 1.12.0-beta.1 on beta channel [skip ci]

## [1.12.0-beta.1](v1.11.1...v1.12.0-beta.1) (2023-07-18)

### 🍕 Features

* view repos on opensauced ([#218](#218)) ([f33d9fd](f33d9fd))

* fix: post highlights from the extension (#223)

* chore(patch): release 1.12.0-beta.2 on beta channel [skip ci]

## [1.12.0-beta.2](v1.12.0-beta.1...v1.12.0-beta.2) (2023-07-19)

### 🐛 Bug Fixes

* post highlights from the extension ([#223](#223)) ([7d91011](7d91011))

* fix: redirect to home page after posting a highlight (#203)

* fix: redirect to home page after posting a highligh

* Added forceRefresh param to force cache invalidation

* chore: default val forceRefresh, typo

---------

Co-authored-by: Anush008 <[email protected]>

* chore(patch): release 1.12.0-beta.3 on beta channel [skip ci]

## [1.12.0-beta.3](v1.12.0-beta.2...v1.12.0-beta.3) (2023-07-20)

### 🐛 Bug Fixes

* redirect to home page after posting a highlight ([#203](#203)) ([4ebd2d1](4ebd2d1))

* fix: view on open sauced location (#225)

* bug-fix: view on open sauced location

* fix: change classname to constant name

* fix: remove unwanted lint changes in constants

* feat: change variable name

* fix: remove element cast

Co-authored-by: Nick Taylor <[email protected]>

---------

Co-authored-by: Nick Taylor <[email protected]>

* chore(patch): release 1.12.0-beta.4 on beta channel [skip ci]

## [1.12.0-beta.4](v1.12.0-beta.3...v1.12.0-beta.4) (2023-07-25)

### 🐛 Bug Fixes

* view on open sauced location ([#225](#225)) ([0ecfef0](0ecfef0))

* feat: Login with pkce (#230)

* chore(minor): release 1.12.0-beta.5 on beta channel [skip ci]

## [1.12.0-beta.5](v1.12.0-beta.4...v1.12.0-beta.5) (2023-07-28)

### 🍕 Features

* Login with pkce ([#230](#230)) ([42e8d4a](42e8d4a))

---------

Co-authored-by: Divyansh Singh <[email protected]>
Co-authored-by: Marcus Chan <[email protected]>
Co-authored-by: Anush008 <[email protected]>
Co-authored-by: Pratham <[email protected]>
Co-authored-by: Nick Taylor <[email protected]>
github-actions bot pushed a commit that referenced this pull request Jul 31, 2023
## [1.12.0](v1.11.1...v1.12.0) (2023-07-31)

### 🍕 Features

* Login with pkce ([#230](#230)) ([42e8d4a](42e8d4a))
* view repos on opensauced ([#218](#218)) ([f33d9fd](f33d9fd))

### 🐛 Bug Fixes

* manual release.yml ([a41de3f](a41de3f))
* post highlights from the extension ([#223](#223)) ([7d91011](7d91011))
* redirect to home page after posting a highlight ([#203](#203)) ([4ebd2d1](4ebd2d1))
* view on open sauced location ([#225](#225)) ([0ecfef0](0ecfef0))
@github-actions
Copy link

🎉 This PR is included in version 1.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants