Skip to content

Commit c2cf667

Browse files
authored
Define moderation statuses in uppercase (#9559)
1 parent 0e0fe69 commit c2cf667

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

h/static/scripts/group-forms/components/GroupModeration.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export type GroupModerationProps = {
1414
export default function GroupModeration({ group }: GroupModerationProps) {
1515
const [filterStatus, setFilterStatus] = useState<
1616
ModerationStatus | undefined
17-
>('pending');
17+
>('PENDING');
1818

1919
return (
2020
<FormContainer>

h/static/scripts/group-forms/components/ModerationStatusSelect.tsx

+5-5
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {
88
Select,
99
} from '@hypothesis/frontend-shared';
1010

11-
export type ModerationStatus = 'pending' | 'approved' | 'denied' | 'spam';
11+
export type ModerationStatus = 'PENDING' | 'APPROVED' | 'DENIED' | 'SPAM';
1212

1313
export type ModerationStatusSelectProps = {
1414
selected?: ModerationStatus;
@@ -21,10 +21,10 @@ type Option = {
2121
};
2222

2323
const options = new Map<ModerationStatus, Option>([
24-
['pending', { label: 'Pending', icon: DottedCircleIcon }],
25-
['approved', { label: 'Approved', icon: CheckAllIcon }],
26-
['denied', { label: 'Denied', icon: RestrictedIcon }],
27-
['spam', { label: 'Spam', icon: CautionIcon }],
24+
['PENDING', { label: 'Pending', icon: DottedCircleIcon }],
25+
['APPROVED', { label: 'Approved', icon: CheckAllIcon }],
26+
['DENIED', { label: 'Denied', icon: RestrictedIcon }],
27+
['SPAM', { label: 'Spam', icon: CautionIcon }],
2828
]);
2929

3030
export default function ModerationStatusSelect({

h/static/scripts/group-forms/components/test/GroupModeration-test.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,11 @@ describe('GroupModeration', () => {
2020
const wrapper = createComponent();
2121
assert.equal(
2222
wrapper.find('ModerationStatusSelect').prop('selected'),
23-
'pending',
23+
'PENDING',
2424
);
2525
});
2626

27-
['approved', 'denied', 'spam'].forEach(newStatus => {
27+
['APPROVED', 'DENIED', 'SPAM'].forEach(newStatus => {
2828
it('changes selected status on ModerationStatusSelect change', () => {
2929
const wrapper = createComponent();
3030

h/static/scripts/group-forms/components/test/ModerationStatusSelect-test.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ describe('ModerationStatusSelect', () => {
2222

2323
[
2424
{ selected: undefined, expectedText: 'All' },
25-
{ selected: 'pending', expectedText: 'Pending' },
26-
{ selected: 'approved', expectedText: 'Approved' },
27-
{ selected: 'denied', expectedText: 'Denied' },
28-
{ selected: 'spam', expectedText: 'Spam' },
25+
{ selected: 'PENDING', expectedText: 'Pending' },
26+
{ selected: 'APPROVED', expectedText: 'Approved' },
27+
{ selected: 'DENIED', expectedText: 'Denied' },
28+
{ selected: 'SPAM', expectedText: 'Spam' },
2929
].forEach(({ selected, expectedText }) => {
3030
it('shows selected option as button content', () => {
3131
const wrapper = createComponent(selected);

0 commit comments

Comments
 (0)