Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Migrate tests from @actions functions to @actions API functions #325

Open
NicoSerranoP opened this issue Sep 5, 2024 · 4 comments
Open

Migrate tests from @actions functions to @actions API functions #325

NicoSerranoP opened this issue Sep 5, 2024 · 4 comments
Labels
good first issue Good for newcomers

Comments

@NicoSerranoP
Copy link
Member

NicoSerranoP commented Sep 5, 2024 •
edited
Loading

Motivation

We are moving all the p0tion coordinator infrastructure to a single server application (API). We replicated some of the functions from @actions to be compatible with this new API. These functions have the same name plus a suffix `API. For example:

isCoordinator
isCoordinatorAPI

We need to write tests for all these new API functions. We can use the existing tests as a guide but make sure to change anything to make it fit.

Tutorial to setup the API Coordinator application
@NicoSerranoP NicoSerranoP added the good first issue Good for newcomers label Sep 5, 2024
@0xgun
Copy link

0xgun commented Oct 1, 2024

Can I work on this ?

@NicoSerranoP
Copy link
Member Author

@0xgun Yes sure! I just updated the issue description with a tutorial to setup The API coordinator in case you need it :)

@0xgun
Copy link

0xgun commented Oct 2, 2024

I'm not able to find the /packages/api directory.

@NicoSerranoP
Copy link
Member Author

Sorry it is in the feat/nico/cfs-in-backend branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NicoSerranoP @0xgun