Add reports management #186

nbsp · 2022-10-21T17:48:09Z

nbsp commented

2022-10-21 17:48:09 +00:00

implements part of #178, other parts will come later

nbsp added 3 commits 2022-10-21 17:48:09 +00:00

4fe11ee378

Add moderation modal

b490ade544

Add API endpoints for reports

ci/woodpecker/pr/woodpecker Pipeline was successful Details

405a60d249

Add reports tab to moderation modal

nbsp changed title from ~~feat/moderation~~ to Add reports management

2022-10-21 17:51:30 +00:00

floatingghost reviewed 2022-11-01 15:26:01 +00:00

floatingghost left a comment

this is pretty ok, but i think we need a few more things here

refetching reports on a timer so we don't need to reload to get more reports
maybe a little badge icon to tell us how many unresolved reports we have sitting around

this is not mandatory but might be nice

1 is sorta needed, the other you can leave out

this is pretty ok, but i think we need a few more things here 1. refetching reports on a timer so we don't need to reload to get more reports 2. maybe a little badge icon to tell us how many unresolved reports we have sitting around - this is not mandatory but might be nice 1 is sorta needed, the other you can leave out

src/components/desktop_nav/desktop_nav.vue

					
				@ -154,0 +160,4 @@

				            fixed-width

				            class="fa-scale-110 fa-old-padding"

				            icon="user-tie"

				            :title="$t('nav.moderation')"

floatingghost commented

2022-11-01 15:22:32 +00:00

this key is undefined in the locale file

nbsp marked this conversation as resolved

src/components/mod_modal/mod_modal_content.vue Outdated

					
				@ -0,0 +13,4 @@

				    >

				      <ReportsTab />

				    </div>

				    <!-- <div -->

floatingghost commented

2022-11-01 15:23:42 +00:00

should this stuff here be here?

it seems to be older test content?

should this stuff here be here? it seems to be older test content?

nbsp commented

2022-11-01 16:29:19 +00:00

oops yeah

nbsp marked this conversation as resolved

nbsp commented

2022-11-01 16:30:09 +00:00

do you think the reports badge should sit on the moderation button in the navbar? because eventually it's gonna be for more than just reports, so i don't know if showing a number there works