Skip to content

CAS-450: Add Custom Strategy Scripts#424

Merged
jbluks merged 16 commits intomainfrom
jonathan/CAS-450
Sep 8, 2022
Merged

CAS-450: Add Custom Strategy Scripts#424
jbluks merged 16 commits intomainfrom
jonathan/CAS-450

Conversation

@jbluks
Copy link
Copy Markdown
Collaborator

@jbluks jbluks commented Aug 31, 2022

Ticket: CAS-450
Sub-Tickets: CAS-458, CAS-459

Description:

  • Enables ability to add a custom strategy that runs a provided script
  • Custom scripts stored in /main/cadence/scripts/custom
  • /main/cadence/scripts/custom/scripts.json provides mapping of script names to files
  • Adds custom_script.go strategy which looks up script provided
  • FE changes that allow for selecting a custom scrip

Screen Shot 2022-08-31 at 11 42 55 AM

t from a list

Screen Shot 2022-08-31 at 11 43 07 AM

Screen Shot 2022-08-31 at 11 43 12 AM

@linear
Copy link
Copy Markdown

linear Bot commented Aug 31, 2022

CAS-450

Copy link
Copy Markdown
Contributor

@germanurrus germanurrus left a comment

Choose a reason for hiding this comment

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

LGTM! (frontend part)

@jbluks jbluks marked this pull request as ready for review September 6, 2022 22:19
Copy link
Copy Markdown
Contributor

@0xmovses 0xmovses left a comment

Choose a reason for hiding this comment

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

time for this to fly the nest!

@jbluks jbluks merged commit 56cfe4d into main Sep 8, 2022
@jbluks jbluks deleted the jonathan/CAS-450 branch September 8, 2022 19:17
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