Skip to content
This repository has been archived by the owner on May 25, 2023. It is now read-only.

fix: able to add the same repo twice #975

Merged
merged 2 commits into from
May 2, 2021

Conversation

farisaziz12
Copy link
Contributor

@farisaziz12 farisaziz12 commented May 2, 2021

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

  • 馃悰 Bug Fix

Description

Adds an error for when you try to add the same repo twice on the dashboard page.

Related Tickets & Documents

Fixes #974

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Screenshot 2021-05-02 at 14 24 40

Added tests?

  • 馃檯 no, because they aren't needed

Added to documentation?

  • 馃檯 no documentation needed

@farisaziz12 farisaziz12 changed the title fix: dd duplicate repo fix: add duplicate repo May 2, 2021
@farisaziz12 farisaziz12 changed the title fix: add duplicate repo fix: able to add the same repo twice May 2, 2021
Copy link
Member

@bdougie bdougie left a comment

Choose a reason for hiding this comment

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

LGTM. Nice catch @farisaziz12. Thanks for the contribution!

@bdougie bdougie merged commit d10d3a2 into open-sauced:main May 2, 2021
@farisaziz12 farisaziz12 deleted the fix/add-duplicate-repo branch May 2, 2021 16:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Can add the same repository multiple times
2 participants