You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/07/25 04:11:13 UTC
[GitHub] [apisix-dashboard] wmdmomo opened a new pull request #1998: feat(plugin): fix UI limit-count
wmdmomo opened a new pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998
Please answer these questions before submitting a pull request, **or your PR will get closed**.
**Why submit this pull request?**
- [ ] Bugfix
- [ ] New feature provided
- [x] Improve performance
- [ ] Backport patches
**What changes will this PR take into?**
The PR will change the plugin under plugin/UI (limit-count), allowing limit-count to dynamically adapt to the back-end json rules.
**Related issues**
fix/resolve #1588
**Checklist:**
- [x] Did you explain what problem does this PR solve? Or what new features have been added?
- [x] Have you added corresponding test cases?
- [x] Have you modified the corresponding document?
- [x] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] wmdmomo commented on a change in pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
wmdmomo commented on a change in pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#discussion_r677193372
##########
File path: web/src/components/Plugin/UI/limit-count.tsx
##########
@@ -149,14 +159,14 @@ const RedisClusterForm: React.FC = () => {
</Form.Item>
<Form.Item {...FORM_ITEM_WITHOUT_LABEL}>
- <Button
+ {fields.length < maxLength?(<Button
Review comment:
Thanks for your suggestion, I have modified it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
βοΈ Deploy Preview for *apisix-dashboard* ready!
π¨ Explore the source changes: fe5e77a6703e2e1493ca76ac9badb3283ea446a8
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60ffb645b1bfd500072b5de1](https://app.netlify.com/sites/apisix-dashboard/deploys/60ffb645b1bfd500072b5de1?utm_source=github&utm_campaign=bot_dl)
π Browse the preview: [https://deploy-preview-1998--apisix-dashboard.netlify.app/](https://deploy-preview-1998--apisix-dashboard.netlify.app/?utm_source=github&utm_campaign=bot_dp)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] commented on pull request #1998: feat(plugin): fix UI limit-count
Posted by GitBox <gi...@apache.org>.
netlify[bot] commented on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
π· Deploy Preview for *apisix-dashboard* processing.
π¨ Explore the source changes: 87ba039a60a472eb6edc1df59790d8fa5231b42f
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60fce45682614600086b7691](https://app.netlify.com/sites/apisix-dashboard/deploys/60fce45682614600086b7691?utm_source=github&utm_campaign=bot_dl)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] liuxiran merged pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
liuxiran merged pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
codecov-commenter edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886308670
# [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#1998](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (fe5e77a) into [master](https://codecov.io/gh/apache/apisix-dashboard/commit/0b5196f0b444dac8acb5cf60635806f024de6a0a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (0b5196f) will **increase** coverage by `0.05%`.
> The diff coverage is `100.00%`.
[](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## master #1998 +/- ##
==========================================
+ Coverage 64.01% 64.06% +0.05%
==========================================
Files 122 122
Lines 3154 3164 +10
Branches 753 755 +2
==========================================
+ Hits 2019 2027 +8
- Misses 1135 1137 +2
```
| Flag | Coverage Ξ | |
|---|---|---|
| frontend-e2e-test | `64.06% <100.00%> (+0.05%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Ξ | |
|---|---|---|
| [web/src/components/Plugin/PluginDetail.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5EZXRhaWwudHN4) | `65.82% <100.00%> (+0.21%)` | :arrow_up: |
| [web/src/components/Plugin/UI/limit-count.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9saW1pdC1jb3VudC50c3g=) | `87.17% <100.00%> (+3.84%)` | :arrow_up: |
| [web/src/components/Plugin/UI/plugin.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9wbHVnaW4udHN4) | `82.60% <100.00%> (ΓΈ)` | |
| [web/src/helpers.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9oZWxwZXJzLnRzeA==) | `68.85% <0.00%> (-3.28%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Ξ = absolute <relative> (impact)`, `ΓΈ = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [0b5196f...fe5e77a](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): fix UI limit-count
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
βοΈ Deploy Preview for *apisix-dashboard* ready!
π¨ Explore the source changes: 42890de006c6536441ebcd7ac8334434fcd828a9
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60fec51886180d000855d7bc](https://app.netlify.com/sites/apisix-dashboard/deploys/60fec51886180d000855d7bc?utm_source=github&utm_campaign=bot_dl)
π Browse the preview: [https://deploy-preview-1998--apisix-dashboard.netlify.app/](https://deploy-preview-1998--apisix-dashboard.netlify.app/?utm_source=github&utm_campaign=bot_dp)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] liuxiran commented on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
liuxiran commented on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-902551567
merged, thanks a lot
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] guoqqqi commented on a change in pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
guoqqqi commented on a change in pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#discussion_r676795398
##########
File path: web/src/components/Plugin/UI/limit-count.tsx
##########
@@ -135,7 +145,7 @@ const RedisClusterForm: React.FC = () => {
</Form.Item>
</Col>
<Col style={{ ...removeBtnStyle, marginLeft: -10 }}>
- {fields.length > 1 ? (
+ {fields.length > minLength ? (
Review comment:
I think it would be better to change this part of the Conditional (ternary) operator to the && operator
##########
File path: web/src/components/Plugin/UI/limit-count.tsx
##########
@@ -149,14 +159,14 @@ const RedisClusterForm: React.FC = () => {
</Form.Item>
<Form.Item {...FORM_ITEM_WITHOUT_LABEL}>
- <Button
+ {fields.length < maxLength?(<Button
Review comment:
ditto
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
βοΈ Deploy Preview for *apisix-dashboard* ready!
π¨ Explore the source changes: d811800f0a127afefb29dbf9fbea2a09080b4904
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/611109e14aeefe0008768f06](https://app.netlify.com/sites/apisix-dashboard/deploys/611109e14aeefe0008768f06?utm_source=github&utm_campaign=bot_dl)
π Browse the preview: [https://deploy-preview-1998--apisix-dashboard.netlify.app/](https://deploy-preview-1998--apisix-dashboard.netlify.app/?utm_source=github&utm_campaign=bot_dp)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
π· Deploy Preview for *apisix-dashboard* processing.
π¨ Explore the source changes: d811800f0a127afefb29dbf9fbea2a09080b4904
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/611109e14aeefe0008768f06](https://app.netlify.com/sites/apisix-dashboard/deploys/611109e14aeefe0008768f06?utm_source=github&utm_campaign=bot_dl)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): fix UI limit-count
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
βοΈ Deploy Preview for *apisix-dashboard* ready!
π¨ Explore the source changes: 87ba039a60a472eb6edc1df59790d8fa5231b42f
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60fce45682614600086b7691](https://app.netlify.com/sites/apisix-dashboard/deploys/60fce45682614600086b7691?utm_source=github&utm_campaign=bot_dl)
π Browse the preview: [https://deploy-preview-1998--apisix-dashboard.netlify.app/](https://deploy-preview-1998--apisix-dashboard.netlify.app/?utm_source=github&utm_campaign=bot_dp)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): fix UI limit-count
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
π· Deploy Preview for *apisix-dashboard* processing.
π¨ Explore the source changes: 42890de006c6536441ebcd7ac8334434fcd828a9
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60fec51886180d000855d7bc](https://app.netlify.com/sites/apisix-dashboard/deploys/60fec51886180d000855d7bc?utm_source=github&utm_campaign=bot_dl)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] codecov-commenter edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
codecov-commenter edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886308670
# [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#1998](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (3e0787c) into [master](https://codecov.io/gh/apache/apisix-dashboard/commit/0b5196f0b444dac8acb5cf60635806f024de6a0a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (0b5196f) will **increase** coverage by `0.09%`.
> The diff coverage is `95.45%`.
> :exclamation: Current head 3e0787c differs from pull request most recent head 42890de. Consider uploading reports for the commit 42890de to get more accurate results
[](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## master #1998 +/- ##
==========================================
+ Coverage 64.01% 64.10% +0.09%
==========================================
Files 122 122
Lines 3154 3165 +11
Branches 753 756 +3
==========================================
+ Hits 2019 2029 +10
- Misses 1135 1136 +1
```
| Flag | Coverage Ξ | |
|---|---|---|
| frontend-e2e-test | `64.10% <95.45%> (+0.09%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Ξ | |
|---|---|---|
| [web/src/components/Plugin/UI/limit-count.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9saW1pdC1jb3VudC50c3g=) | `85.00% <94.44%> (+1.66%)` | :arrow_up: |
| [web/src/components/Plugin/PluginDetail.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5EZXRhaWwudHN4) | `65.82% <100.00%> (+0.21%)` | :arrow_up: |
| [web/src/components/Plugin/UI/plugin.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9wbHVnaW4udHN4) | `82.60% <100.00%> (ΓΈ)` | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Ξ = absolute <relative> (impact)`, `ΓΈ = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [0b5196f...42890de](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] netlify[bot] edited a comment on pull request #1998: feat(plugin): allowing limit-count to dynamically adapt to the BE rules
Posted by GitBox <gi...@apache.org>.
netlify[bot] edited a comment on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886144035
π· Deploy Preview for *apisix-dashboard* processing.
π¨ Explore the source changes: fe5e77a6703e2e1493ca76ac9badb3283ea446a8
π Inspect the deploy log: [https://app.netlify.com/sites/apisix-dashboard/deploys/60ffb645b1bfd500072b5de1](https://app.netlify.com/sites/apisix-dashboard/deploys/60ffb645b1bfd500072b5de1?utm_source=github&utm_campaign=bot_dl)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [apisix-dashboard] codecov-commenter commented on pull request #1998: feat(plugin): fix UI limit-count
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #1998:
URL: https://github.com/apache/apisix-dashboard/pull/1998#issuecomment-886308670
# [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#1998](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (87ba039) into [master](https://codecov.io/gh/apache/apisix-dashboard/commit/0b5196f0b444dac8acb5cf60635806f024de6a0a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (0b5196f) will **increase** coverage by `0.03%`.
> The diff coverage is `95.45%`.
[](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## master #1998 +/- ##
==========================================
+ Coverage 64.01% 64.04% +0.03%
==========================================
Files 122 122
Lines 3154 3165 +11
Branches 753 756 +3
==========================================
+ Hits 2019 2027 +8
- Misses 1135 1138 +3
```
| Flag | Coverage Ξ | |
|---|---|---|
| frontend-e2e-test | `64.04% <95.45%> (+0.03%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
| [Impacted Files](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Ξ | |
|---|---|---|
| [web/src/components/Plugin/UI/limit-count.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9saW1pdC1jb3VudC50c3g=) | `85.00% <94.44%> (+1.66%)` | :arrow_up: |
| [web/src/components/Plugin/PluginDetail.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5EZXRhaWwudHN4) | `65.82% <100.00%> (+0.21%)` | :arrow_up: |
| [web/src/components/Plugin/UI/plugin.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9wbHVnaW4udHN4) | `82.60% <100.00%> (ΓΈ)` | |
| [web/src/helpers.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/1998/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9oZWxwZXJzLnRzeA==) | `68.85% <0.00%> (-3.28%)` | :arrow_down: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Ξ = absolute <relative> (impact)`, `ΓΈ = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [0b5196f...87ba039](https://codecov.io/gh/apache/apisix-dashboard/pull/1998?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org