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.

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

feature(api): added api for faucet to allocate fund to user #346

Merged
merged 5 commits into from
Nov 8, 2023

Conversation

fullstackninja864
Copy link
Member

What this PR does / why we need it:

Which issue(s) does this PR fixes?:

Fixes #

Additional comments?:

Developer Checklist:

  • Read your code changes at least once
  • Your UI implementation visually matched the rendered design*
  • Unit tests*
  • Added e2e tests*

@fullstackninja864 fullstackninja864 self-assigned this Nov 7, 2023
@github-actions github-actions bot added the kind/feature New feature request label Nov 7, 2023
apps/server/src/faucet/FaucetController.ts Outdated Show resolved Hide resolved
apps/server/src/faucet/FaucetController.ts Outdated Show resolved Hide resolved
@pierregee pierregee merged commit 23175ef into lyka/api-setup Nov 8, 2023
2 checks passed
@fullstackninja864 fullstackninja864 deleted the harsh/faucet branch November 8, 2023 03:12
pierregee pushed a commit that referenced this pull request Nov 8, 2023
* feat(server): nestjs setup

* remove nvmrc

* feature(api): added api for faucet to allocate fund to user (#346)

* feature(api): added api for faucet to allocate fund to user

* added e2e test

* fixed pr comments

* added e2e for ratelimiting

* added invalid address test case

* fix lint

* fix typo

* fix lint

* fix ci

* fix ci

* fix ci

* fix ci

* fix ci

* fix ci

* fix ci

* fix ci

* fix ci-e2e

* fix ci-e2e

---------

Co-authored-by: Harsh R <53080940+fullstackninja864@users.noreply.github.com>
Co-authored-by: pierregee <pierre@cakedefi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants