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

feat: using denylist from edge-gateway #126

Merged
merged 1 commit into from
Dec 20, 2022
Merged

Conversation

jsdevel
Copy link
Contributor

@jsdevel jsdevel commented Dec 12, 2022

No description provided.

@jsdevel jsdevel force-pushed the feat-68-denylist-items branch 5 times, most recently from d20545d to 76601dc Compare December 12, 2022 22:59
Copy link
Contributor

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

@jsdevel good work. Minimal changes needed to land this :)

packages/edge-gateway/src/gateway.js Outdated Show resolved Hide resolved
packages/edge-gateway/src/gateway.js Outdated Show resolved Hide resolved
packages/edge-gateway/src/gateway.js Outdated Show resolved Hide resolved
@jsdevel jsdevel force-pushed the feat-68-denylist-items branch 22 times, most recently from 4efac23 to 9b83799 Compare December 19, 2022 22:20
Copy link
Contributor

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

LGTM

@jsdevel jsdevel merged commit 22c0123 into main Dec 20, 2022
@jsdevel jsdevel deleted the feat-68-denylist-items branch December 20, 2022 16:32
vasco-santos pushed a commit that referenced this pull request Dec 21, 2022
🤖 I have created a release *beep* *boop*
---


## 1.0.0 (2022-12-20)


### Features

* adding denylist package (closes
[#68](#68))
([ad93e85](ad93e85))
* using denylist from edge-gateway
([#126](#126))
([22c0123](22c0123))


### Bug Fixes

* denylist tests and logging
([82e3eaa](82e3eaa))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
vasco-santos pushed a commit that referenced this pull request Dec 21, 2022
🤖 I have created a release *beep* *boop*
---


##
[1.2.0](cid-verifier-v1.1.0...cid-verifier-v1.2.0)
(2022-12-20)


### Features

* using denylist from edge-gateway
([#126](#126))
([22c0123](22c0123))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
vasco-santos pushed a commit that referenced this pull request Dec 21, 2022
🤖 I have created a release *beep* *boop*
---


##
[1.9.0](edge-gateway-v1.8.0...edge-gateway-v1.9.0)
(2022-12-21)


### Features

* using denylist from edge-gateway
([#126](#126))
([22c0123](22c0123))


### Bug Fixes

* add kv bindings for denylist in edge gateway
([#136](#136))
([97dcbe2](97dcbe2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants