openedx / frontend-app-discussions

A React-based micro frontend for the Open edX discussion forums.
GNU Affero General Public License v3.0
6 stars 62 forks source link

feat: integrated backend discussions restriction with MFE #529

Closed ayesha-waris closed 1 year ago

ayesha-waris commented 1 year ago

INF-828

Description

Integrated already build UI with the API change in INF-826

Post-merge Checklist

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.03 :tada:

Comparison is base (c8500a0) 91.82% compared to head (0891e76) 91.86%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #529 +/- ## ========================================== + Coverage 91.82% 91.86% +0.03% ========================================== Files 170 170 Lines 3476 3468 -8 Branches 905 904 -1 ========================================== - Hits 3192 3186 -6 + Misses 265 263 -2 Partials 19 19 ``` | [Impacted Files](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | Coverage Δ | | |---|---|---| | [src/discussions/data/slices.js](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2RhdGEvc2xpY2VzLmpz) | `83.33% <ø> (ø)` | | | [src/discussions/messages.js](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL21lc3NhZ2VzLmpz) | `100.00% <ø> (ø)` | | | [src/discussions/utils.js](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL3V0aWxzLmpz) | `84.21% <ø> (-1.51%)` | :arrow_down: | | [src/discussions/common/ActionsDropdown.jsx](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2NvbW1vbi9BY3Rpb25zRHJvcGRvd24uanN4) | `87.50% <100.00%> (+3.12%)` | :arrow_up: | | [src/discussions/common/HoverCard.jsx](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2NvbW1vbi9Ib3ZlckNhcmQuanN4) | `86.20% <100.00%> (ø)` | | | [src/discussions/data/hooks.js](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2RhdGEvaG9va3MuanM=) | `87.17% <100.00%> (-0.11%)` | :arrow_down: | | [src/discussions/data/selectors.js](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2RhdGEvc2VsZWN0b3JzLmpz) | `100.00% <100.00%> (ø)` | | | [...c/discussions/discussions-home/DiscussionsHome.jsx](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2Rpc2N1c3Npb25zLWhvbWUvRGlzY3Vzc2lvbnNIb21lLmpzeA==) | `96.00% <100.00%> (ø)` | | | [.../discussions-home/DiscussionsRestrictionBanner.jsx](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL2Rpc2N1c3Npb25zLWhvbWUvRGlzY3Vzc2lvbnNSZXN0cmljdGlvbkJhbm5lci5qc3g=) | `88.88% <100.00%> (ø)` | | | [...iscussions/post-comments/comments/CommentsView.jsx](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx#diff-c3JjL2Rpc2N1c3Npb25zL3Bvc3QtY29tbWVudHMvY29tbWVudHMvQ29tbWVudHNWaWV3LmpzeA==) | `85.71% <100.00%> (ø)` | | | ... and [2 more](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx) | | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/openedx/frontend-app-discussions/pull/529/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.