Skip to content

[DT-3158] feat: import from github in reuse existing slices dialog#1754

Merged
jomifepe merged 36 commits intojp/import-slice-basefrom
jp/import-slices-2
Dec 29, 2025
Merged

[DT-3158] feat: import from github in reuse existing slices dialog#1754
jomifepe merged 36 commits intojp/import-slice-basefrom
jp/import-slices-2

Conversation

@jomifepe
Copy link
Collaborator

@jomifepe jomifepe commented Dec 18, 2025

Resolves: DT-3158

Description

  1. Add two tabs to the new dialog: Local slices and Library slices.
  2. Add the existing Reuse and existing slice functionality to the Local slice tab and the new GitHub importing feature in the Library slices tab.
  3. Use the new modal with import from GitHub in place of the Reuse an existing slice one

Preview

Clipboard-20251218-180218-916.mp4

How to QA 1

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

@jomifepe jomifepe self-assigned this Dec 18, 2025
@linear
Copy link

linear bot commented Dec 18, 2025

@vercel
Copy link

vercel bot commented Dec 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
slice-machine Ready Ready Preview Dec 24, 2025 3:52pm

@jomifepe jomifepe marked this pull request as ready for review December 18, 2025 18:06
@jomifepe jomifepe requested a review from a team as a code owner December 18, 2025 18:06
refactor: share types
@jomifepe jomifepe requested a review from xrutayisire December 22, 2025 17:18
fix: reset tab on close

refactor: match onSuccess types to the previous ones
refactor: renaming
refactor: remove unnecessary mock updating code
@jomifepe jomifepe merged commit 4348ed2 into jp/import-slice-base Dec 29, 2025
33 checks passed
@jomifepe jomifepe deleted the jp/import-slices-2 branch December 29, 2025 11:59
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.

3 participants