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/add reset deploy #879 #1074

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

HatemLamia
Copy link
Contributor

Description

This pull request implements the ability to cancel ongoing deployments in the Dev Center, addressing issue #879. The changes enhance the user experience by allowing users to cancel deployments mid-progress, providing feedback, and resetting the interface appropriately.

Changes Implemented

UI Updates

  • Cancel Button:

    • Added a "✕" cancel button alongside the deployment progress indicator (e.g., "Deploying (X%)").
    • The button is prominently displayed to ensure usability.
  • Feedback Messages:

    • Added "Cancelling..." message when a user initiates a cancellation.
    • Feedback is provided in scenarios where cancellation is not possible (e.g., near completion or during critical operations).

Functional Updates

  • Cancellation Handling:
    • Cancel any ongoing file uploads or transfers when the cancel button is clicked.
    • Clean up any partially uploaded files in the app directory.
    • Reset the deployment drop area to its initial state.

Code Updates

All updates were made in dev-center.js and 'style.css'

Screenshots

image

@jelveh
Copy link
Contributor

jelveh commented Dec 27, 2024

Hi there, this removes the "file association tagging". Could you make sure you're not removing existing features?

@HatemLamia
Copy link
Contributor Author

Hi @jelveh ,
Thank you for your feedback. I believe I haven’t removed any existing features. Could you please clarify what you mean? Thank you!

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.

2 participants