research-software-directory / RSD-as-a-service

This repo contains the new RSD-as-a-service implementation
https://research.software
27 stars 14 forks source link

1294 categories for organisations #1315

Closed dmijatovic closed 4 weeks ago

dmijatovic commented 1 month ago

Categories for organisations

Closes #1294

Changes proposed in this pull request:

How to test:

Example organisation categories

image

Example adding organisation categories to software or project

image

Example organisation categories on project page (bottom of side panel)

image

PR Checklist:

jmaassen commented 1 month ago

I seem to have run into a bug. This are the steps I took

Next added project:

Project page of "test" looks as expected

Next added another project:

Project page of "test2" looks as expected

But the categories of project "test" have now disappeared?

dmijatovic commented 1 month ago

@jmaassen Well spotted! I have fixed bug. The PR is (again) ready for testing.

jmaassen commented 1 month ago

During the demo for @bakhshir I ran into an issue with being able to add a category as a organization maintainer. What I did is the following:

I then get a error about row level security:

Screenshot 2024-10-16 at 14-51-46 Netherlands eScience Center Research Software Directory

ewan-escience commented 1 month ago

Does that error still happen on the latest changes? I pushed some changes to the row level security earlier today.

dmijatovic commented 1 month ago

@jmaassen I fixed the modal error Ewan noticed. Can you test again?

jmaassen commented 1 month ago

The issue is gone now, and everything works as expected.

sonarcloud[bot] commented 1 month ago

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

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
14.6% Duplication on New Code

See analysis details on SonarCloud

sonarcloud[bot] commented 1 month ago

Quality Gate Failed Quality Gate failed for 'rsd-frontend'

Failed conditions
5.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud