Skip to content
This repository was archived by the owner on Dec 28, 2024. It is now read-only.

Conversation

@aematei
Copy link
Collaborator

@aematei aematei commented Oct 28, 2023

Description

Setup abstract classes/interfaces for frontend typescript logic

Subsequent # LL-248

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist (when relevant):

  • My code follows the style guidelines of this project
    TypeScript
    Schema
  • I have included a reviewer on the pull request [Replace this text with the reviewer]
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@pogi7 pogi7 marked this pull request as ready for review October 28, 2023 19:20
@pogi7 pogi7 marked this pull request as draft October 28, 2023 19:23
@aematei aematei self-assigned this Oct 29, 2023
@aematei
Copy link
Collaborator Author

aematei commented Feb 20, 2024

@pogi7 Can you help me out with something? I'm trying to use Zustand to make a hook to store state of a todo list for my example service. I'm getting an error in src/components/ExampleServicePage.tsx that I can't seem to figure out. I'm trying to bind my service API method service.getAll() to the refresh setter in my hook. Can you take a look when you have a chance? We can also tag up if you'd prefer.

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants