Skip to content

Organization-level Domain Tags#3879

Merged
lcampbell2 merged 25 commits intomasterfrom
user-generated-tags
Sep 28, 2022
Merged

Organization-level Domain Tags#3879
lcampbell2 merged 25 commits intomasterfrom
user-generated-tags

Conversation

@lcampbell2
Copy link
Copy Markdown
Collaborator

@lcampbell2 lcampbell2 commented Jul 28, 2022

Admins can now add pre-determined tag labels to each domain in their organization, which are displayed on the organization details page, and the admin page

Org Details Page:
Screenshot from 2022-07-28 14-46-33

Admin Page:
Screenshot from 2022-07-28 14-45-55

Screenshot from 2022-07-28 14-46-07

closes #3667

@lcampbell2 lcampbell2 requested a review from FestiveKyle July 28, 2022 17:52
description: '',
fields: () => ({
en: {
type: GraphQLString,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe make non-nullable?

description: 'The English translation of the label.',
},
fr: {
type: GraphQLString,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non-nullable?

Comment on lines +76 to +82
let tags
if (typeof args.tags !== 'undefined') {
tags = args.tags
} else {
tags = []
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This defaults tags to an empty array but the updateDomain mutation defaults tags to null


let tags
if (typeof args.tags !== 'undefined') {
tags = args.tags
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably cleanse input to be safe


let tags
if (typeof args.tags !== 'undefined') {
tags = args.tags
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably cleanse input to be safe

borderWidth="1px"
borderColor="gray.900"
>
<TagLabel mx="auto">{label[locale]}</TagLabel>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

locale not required if handled at loader.

)
}
AdminDomainCard.propTypes = { url: string }
AdminDomainCard.propTypes = { url: string, tags: array, locale: string }
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

locale not required if handled at loader.

borderWidth="1px"
borderColor="gray.900"
>
<TagLabel mx="auto">{label[i18n.locale]}</TagLabel>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

locale not required if handled at loader.

Comment on lines +533 to +538
claimTags {
label {
en
fr
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

claimTags if translation handled at loader.

Comment on lines +625 to +630
claimTags {
label {
en
fr
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

claimTags if translation handled at loader.

@lcampbell2 lcampbell2 requested a review from FestiveKyle August 16, 2022 19:35
@lcampbell2 lcampbell2 merged commit 127e4b8 into master Sep 28, 2022
@lcampbell2 lcampbell2 deleted the user-generated-tags branch September 28, 2022 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Develop tagging capabilities for orgs and domains

2 participants