Dsuh avatar selection bug #1162
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dvdhs
force-pushed
the
dsuh-avatar-selection-bug
branch
from
9f5a509 to
470f917
Compare
dvdhs
added a commit
that referenced
this pull request
Jul 25, 2023
* polygon url * polygon url (#1163) Co-authored-by: David Suh <[email protected]> * Dsuh avatar selection bug (#1162) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * Dsuh 943 bat gating (#1166) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader * adding tooltip * Dsuh polygon gating (#1170) * polygon url * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * added polygon to gating --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
mrose17
added a commit
that referenced
this pull request
Jul 25, 2023
* BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Polygon Gating (#1168) * polygon url * polygon url (#1163) Co-authored-by: David Suh <[email protected]> * Dsuh avatar selection bug (#1162) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * Dsuh 943 bat gating (#1166) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader * adding tooltip * Dsuh polygon gating (#1170) * polygon url * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * added polygon to gating --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
hadiamjad
added a commit
that referenced
this pull request
Jul 26, 2023
* Polygon Gating (#1168) * polygon url * polygon url (#1163) Co-authored-by: David Suh <[email protected]> * Dsuh avatar selection bug (#1162) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * Dsuh 943 bat gating (#1166) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader * adding tooltip * Dsuh polygon gating (#1170) * polygon url * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * added polygon to gating --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * updated sol gating and rebased with main --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
mrose17
added a commit
that referenced
this pull request
Aug 1, 2023
* BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Polygon Gating (#1168) * polygon url * polygon url (#1163) Co-authored-by: David Suh <[email protected]> * Dsuh avatar selection bug (#1162) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * Dsuh 943 bat gating (#1166) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader * adding tooltip * Dsuh polygon gating (#1170) * polygon url * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * added polygon to gating --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
mrose17
added a commit
that referenced
this pull request
Aug 1, 2023
* BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Polygon Gating (#1168) * polygon url * polygon url (#1163) Co-authored-by: David Suh <[email protected]> * Dsuh avatar selection bug (#1162) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * Dsuh 943 bat gating (#1166) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader * adding tooltip * Dsuh polygon gating (#1170) * polygon url * Avatar selection bug fix for Web3 Brave Talk (#1164) * adding inital fix * changing NFT border selection from imageurl comp to id comp * chaged indexFind * debug messages in prod only --------- Co-authored-by: David Suh <[email protected]> * BAT gating translations (#1167) * first mockup on dev2 * remove cluttered talk, add request payload * change permission name from BAT-gating to balance as per spec sheet * Renovate/all minor patch (#1062) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Renovate/node 18.x (#1068) * Use renovate for dependency updates (#1054) 1. Disable dependabot for normal dependency updates. Note the dependabot PRs will need to be manually closed, and the "security" dependabot PRs will continue. 2. Use renovateapp instead. The renovate configuration included here is based on that from ads-serve, and: a. waits for updates to have been published for 4 days before raising a PR - this both reduces churn and also avoids the 3 day period where npm-published entries can be "unpublished" b. combines all minor and patch updates into a single PR * removed as *[bot] is not accepted by AWS (#1055) Signed-off-by: Ahmed Kamal <[email protected]> * Update all non-major dependencies (#1057) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Pin dependencies (#1056) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1063) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Node.js to v18.16.1 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * aesthetic changes * updating default gated value to 1 wei * adding type/enum safety and removing needless futureproofing, using translation rather than string literals * simple change for dev2 compat * fix error message + typo * adding error text for no BAT in BAT-gating * fixed some optional stuff * remove unsused prop in NonExpandablePanel * adding japanese translations --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * update NFT subheader --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * added polygon to gating --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Marshall T. Rose <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update all non-major dependencies (#1180) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update dependency typescript to v5 --------- Signed-off-by: Ahmed Kamal <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: David Suh <[email protected]> Co-authored-by: Graham Tackley <[email protected]> Co-authored-by: Ahmed Kamal <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.