Skip to content

🔨 Switch DeleteSandboxModal to use useOvermind #2524

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
Merged
merged 2 commits into from
Sep 27, 2019
Merged

🔨 Switch DeleteSandboxModal to use useOvermind #2524

merged 2 commits into from
Sep 27, 2019

Conversation

@MichaelDeBoey
Copy link
Contributor

@MichaelDeBoey MichaelDeBoey commented Sep 26, 2019

No description provided.

@vercel
Copy link

vercel bot commented Sep 26, 2019 •
edited
Loading

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://codesandbox-cl-git-fork-michaeldeboey-overmind-deletesan-c451f6.codesandbox1.now.sh

@vercel vercel bot temporarily deployed to staging September 26, 2019 13:01 Inactive
@vercel vercel bot temporarily deployed to staging September 26, 2019 14:02 Inactive
christianalfoni
christianalfoni approved these changes Sep 26, 2019
View reviewed changes
Copy link
Contributor

@christianalfoni christianalfoni left a comment

Choose a reason for hiding this comment

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

Great stuff! :)

@SaraVieira SaraVieira merged commit e41965b into codesandbox:master Sep 27, 2019
@MichaelDeBoey MichaelDeBoey deleted the overmind/DeleteSandboxModal branch September 27, 2019 13:09
yeion7 pushed a commit to yeion7/codesandbox-client that referenced this pull request Sep 30, 2019
@MichaelDeBoey @yeion7
🔨 Switch DeleteSandboxModal to use useOvermind (codesandbox#2524)
e7daf00 
* 🔨 Switch DeleteSandboxModal to use useOvermind

* Fix types
@Saeris Saeris added 🔨 Refactor 🧠 Overmind Indicates that this is related to the app's State Management labels Oct 3, 2019
@adamlaska adamlaska mentioned this pull request Mar 29, 2024
Open
@CompuIves CompuIves mentioned this pull request Mar 29, 2024
Closed
@vutting4221 vutting4221 mentioned this pull request Mar 29, 2024
Open
@uniibu uniibu mentioned this pull request Mar 29, 2024
Open
@adamlaska adamlaska mentioned this pull request Mar 29, 2024
Open
@CharaD7 CharaD7 mentioned this pull request Mar 29, 2024
Open
@baophucct baophucct mentioned this pull request Mar 29, 2024
Open
@Bhanditz Bhanditz mentioned this pull request Mar 29, 2024
Open
@Matthelonianxl Matthelonianxl mentioned this pull request Mar 29, 2024
Open
@saurabharch saurabharch mentioned this pull request Mar 30, 2024
Open
@adamlaska adamlaska mentioned this pull request Mar 30, 2024
Open
@ajesse11x ajesse11x mentioned this pull request Mar 30, 2024
Open
@uniibu uniibu mentioned this pull request May 13, 2024
Open
@adamlaska adamlaska mentioned this pull request May 14, 2024
Open
@CompuIves CompuIves mentioned this pull request May 14, 2024
Open
@CompuIves CompuIves mentioned this pull request May 22, 2024
Open
@adamlaska adamlaska mentioned this pull request May 23, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@christianalfoni christianalfoni christianalfoni approved these changes

+1 more reviewer

@Saeris Saeris Saeris approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

🧠 Overmind Indicates that this is related to the app's State Management 🔨 Refactor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MichaelDeBoey @Saeris @christianalfoni @SaraVieira