Skip to content

[_]: feat/local storage, config and sdk services#4

Merged
xabg2 merged 4 commits intomasterfrom
feature/services-and-its-tests
Mar 4, 2026
Merged

[_]: feat/local storage, config and sdk services#4
xabg2 merged 4 commits intomasterfrom
feature/services-and-its-tests

Conversation

@xabg2
Copy link
Contributor

@xabg2 xabg2 commented Mar 4, 2026

New services have been added:

  • Config service: manages environment variables in a centralized way.
  • Local storage service: provides a layer over the native localStorage API, so we can save or retrieve the user, token, or mnemonic with simple function calls instead of manipulating localStorage manually.
  • SDK service: manages SDK instances for auth, user, and payments, providing a single point of access.
  • Payment service: handles payment-related logic, such as retrieving the user’s tier and subscription details.

Besides:

  • Basic Vitest setup has been added, along with tests covering the services mentioned above.
  • Basic CI workflow added to automatically run tests and build when a PR is opened targeting master.

@xabg2 xabg2 requested a review from larryrider March 4, 2026 11:45
@xabg2 xabg2 self-assigned this Mar 4, 2026
@xabg2 xabg2 added the enhancement New feature or request label Mar 4, 2026

- run: yarn install --frozen-lockfile
- run: yarn test
- run: yarn build

Choose a reason for hiding this comment

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

maybe it makes more sense to run the build before the tests 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mmmm, yeah, could be.

@xabg2 xabg2 merged commit 4284932 into master Mar 4, 2026
1 check passed
@xabg2 xabg2 deleted the feature/services-and-its-tests branch March 4, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants