Skip to content

feat: remove deploy and revert changes buttons#493

Merged
ferruhcihan merged 7 commits intomainfrom
APL-241
Jan 16, 2025
Merged

feat: remove deploy and revert changes buttons#493
ferruhcihan merged 7 commits intomainfrom
APL-241

Conversation

@ferruhcihan
Copy link
Collaborator

@ferruhcihan ferruhcihan commented Jan 7, 2025

Implement

https://jira.linode.com/browse/APL-241

Description

This PR removes the deploy and revert changes buttons and updates the session provider to improve snack notification messages.
Is paired with: linode/apl-api#608

@dennisvankekem
Copy link
Collaborator

Code looks clean

@ElderMatt ElderMatt self-assigned this Jan 8, 2025
@CasLubbers
Copy link
Contributor

Tested and it works but adding something is pretty slow... If you are a bit impatient and click to other pages it looks a bit weird. We could maybe add an loading thing on the whole page so you don't go to other pages.
Other option is to see what is taking so long and make the requests a bit faster.

@j-zimnowoda
Copy link
Contributor

Tested and it works but adding something is pretty slow... If you are a bit impatient and click to other pages it looks a bit weird. We could maybe add an loading thing on the whole page so you don't go to other pages. Other option is to see what is taking so long and make the requests a bit faster.

Good point. We should block user actions on page until there is a response from server

@ferruhcihan ferruhcihan enabled auto-merge (squash) January 16, 2025 11:08
@ferruhcihan ferruhcihan merged commit 91ed8a6 into main Jan 16, 2025
2 checks passed
@ferruhcihan ferruhcihan deleted the APL-241 branch January 16, 2025 11:11
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.

5 participants