Skip to content

Manual deployment#19

Open
AlexRodwell wants to merge 27 commits intomainfrom
wip
Open

Manual deployment#19
AlexRodwell wants to merge 27 commits intomainfrom
wip

Conversation

@AlexRodwell
Copy link
Owner

@AlexRodwell AlexRodwell commented Aug 25, 2024

Make manual deployment work (mostly), still WIP.

TODO:

  • Ensure cards can be selected/toggled when in manual deployment mode (they currently don't change image if in manual)
  • Fix global settings toggle states not dispatching/deploying on change

feat: add disable state to deploy button
feat: add conditional text based on `$deploy.proceed` value
feat: passed in deploy_data function as prop to dashboard's `<Header>` component, to be called by deploy button, if not disabled
refactor: update `store_data` function to only `deploy_data` if not manual deployment
refactor: move `$deploy.proceed = false` to outside the `forEach` loop on `deploy_data` function, was uncessariyl assigning false several times
…d buttons without an after icon to be longer than the inner content
…` as imported assets, as we want to preload these in the `<head>`
@AlexRodwell AlexRodwell marked this pull request as ready for review August 25, 2024 19:41
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.

1 participant