Skip to content

Conversation

@JuliRossi
Copy link
Collaborator

@JuliRossi JuliRossi commented Oct 31, 2025

Purpose

We want to use modern app actions parameterSchema and resultSchema for the slack app action.

Approach

  • Updated the readme with information to update the dockerfile and serveless node version to 18 to be able to run it locally
  • Added manifest to slack app
  • Add script to upsert actions from manifest
  • Modify the controller => Now it returns what the post actually returns

Did some clean up:

  • Fixed some old tests that were failing =>
    - When running all tests, a controller and repository were having conflicts because we didn't restore the mock.
    - Controller type issue with undefined values

Testing steps

As the app action is only used in Workflows, I created a button to run the action with the new format . This video shows that aproach. We also updated the controller tests
What errors look like:

Grabacion.de.pantalla.2025-10-31.a.la.s.4.25.27.p.m.mov

@JuliRossi JuliRossi changed the title Add manifest to slack Add manifest to slack [] Oct 31, 2025
@JuliRossi JuliRossi marked this pull request as ready for review October 31, 2025 17:50
@JuliRossi JuliRossi requested a review from a team as a code owner October 31, 2025 17:50
@JuliRossi JuliRossi merged commit edfe8fa into master Nov 3, 2025
17 of 18 checks passed
@JuliRossi JuliRossi deleted the add-manifest-to-slack branch November 3, 2025 13:18
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.

3 participants