Skip to content

ENG-54 Add Click Handler to Buttons#1838

Merged
nolanpro merged 10 commits into
developfrom
feature/ENG-54
Jul 25, 2025
Merged

ENG-54 Add Click Handler to Buttons#1838
nolanpro merged 10 commits into
developfrom
feature/ENG-54

Conversation

@caleeli

@caleeli caleeli commented Jul 22, 2025

Copy link
Copy Markdown
Collaborator

As a Designer

I want a configurable JavaScript handler property on all current available button components

So that I can run arbitrary logic (e.g. save, rollback or delete data) with full access to the current screen’s data—and run it only when the user actually clicks the button.

Screencast.from.22-07-25.12.51.18.webm

Related Tickets & Packages

ci:deploy

@caleeli caleeli requested a review from rodriquelca July 22, 2025 16:52
@caleeli

caleeli commented Jul 22, 2025

Copy link
Copy Markdown
Collaborator Author
Screencast.from.22-07-25.12.51.18.webm

@caleeli caleeli changed the base branch from develop to feature/ENG-51 July 22, 2025 16:57
@cypress

cypress Bot commented Jul 22, 2025

Copy link
Copy Markdown

screen-builder    Run #1919

Run Properties:  status check passed Passed #1919  •  git commit 8dccf0ccdc: ENG-54 Add Click Handler to Buttons
Project screen-builder
Branch Review feature/ENG-54
Run status status check passed Passed #1919
Run duration 42m 28s
Commit git commit 8dccf0ccdc: ENG-54 Add Click Handler to Buttons
Committer David Callizaya
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 374
View all changes introduced in this branch ↗︎

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@caleeli caleeli requested a review from julceslauhub July 23, 2025 13:05
@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@julceslauhub julceslauhub left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@caleeli caleeli changed the base branch from feature/ENG-51 to develop July 25, 2025 12:15
@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-2ba0b10c63.engk8s.processmaker.net

@processmaker-sonarqube

processmaker-sonarqube Bot commented Jul 25, 2025

Copy link
Copy Markdown

@nolanpro nolanpro merged commit 95fa5f0 into develop Jul 25, 2025
46 of 50 checks passed
@nolanpro nolanpro deleted the feature/ENG-54 branch July 25, 2025 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants