Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve software categories #1143

Merged
merged 7 commits into from
Jun 4, 2024
Merged

improve software categories #1143

merged 7 commits into from
Jun 4, 2024

Conversation

fembau
Copy link
Contributor

@fembau fembau commented Mar 19, 2024

How to test:

see #984

@fembau fembau force-pushed the improve-software-categories branch from b77b4f1 to 6b520c0 Compare March 19, 2024 13:54
Copy link

Quality Gate Passed Quality Gate passed for 'rsd-database'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Categories which are marked as `is_highlight` are visually brought out.
In the software edit view they get an extra dropdown menu and in the
software view they have an extra section in the main content area.

Categories on different levels in the hierarchy could have different
extra properties. These are now aggregated in one database column
`properties` of type JSONB.
@fembau fembau force-pushed the improve-software-categories branch from 6b520c0 to afff730 Compare May 3, 2024 09:24
fembau added 3 commits May 3, 2024 11:45
@fembau
Copy link
Contributor Author

fembau commented Jun 3, 2024

Implementation of table representation is moved into another PR to speed up community feature PR

@dmijatovic
Copy link
Contributor

@fembau To fix failing unit test please add empty categories array to frontend/__tests__/__mocks__/softwareIndexData.js

const softwarePageData = {
  slug: 'test-slug',
  software: softwareItem,
  citationInfo: SoftwareCitationItems,
  keywords: apiKeywordsBySoftware,
  licenseInfo,
  repositoryInfo: repoInfo,
  mentions: apiMentions,
  testimonials: apiTestimonial,
  contributors: apiContributors,
  relatedTools: apiRelatedSoftware,
  relatedProjects: apiRelatedProjects,
  isMaintainer: false,
  organisations: apiOrganisationsOfSoftware,
 // add categories to mock data
  categories:[]
}

@fembau fembau marked this pull request as ready for review June 3, 2024 13:10
@fembau fembau requested review from dmijatovic and cmeessen and removed request for dmijatovic June 3, 2024 13:10
Copy link
Contributor

@cmeessen cmeessen left a comment

Choose a reason for hiding this comment

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

Great work, thank you 👍

Copy link
Contributor

@dmijatovic dmijatovic left a comment

Choose a reason for hiding this comment

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

Nice work! I placed few minor suggestions based on sonar cloud advice.

Copy link

sonarqubecloud bot commented Jun 4, 2024

Quality Gate Passed Quality Gate passed for 'rsd-database'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

sonarqubecloud bot commented Jun 4, 2024

@fembau fembau merged commit 365d2c6 into main Jun 4, 2024
6 checks passed
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.

3 participants