Skip to content

Conversation

@jessegeens
Copy link
Contributor

@jessegeens jessegeens commented Aug 11, 2025

This PR adds support for:

  • displaying description (called readme in the backend), subtitle (called description in the backend) and thumbnail of a space.
  • updating the thumbnail and readme (support for the description has also been added to the CS3API, but not used at the moment as we don't support users updating it)

This required:

Additionally, the projects catalogue now directly implements the methods of the spaces registry.

@update-docs
Copy link

update-docs bot commented Aug 11, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@jessegeens jessegeens force-pushed the fix/spaces-image branch 29 times, most recently from 279a894 to ef870bc Compare August 12, 2025 12:21
@rodcoffani
Copy link
Contributor

rodcoffani commented Aug 13, 2025

Related fix in front-end: cernbox/web/pull/206

Copy link
Contributor

@diocas diocas left a comment

Choose a reason for hiding this comment

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

Just some quick initial comments. To be continued.... :)

@jessegeens jessegeens force-pushed the fix/spaces-image branch 7 times, most recently from f5c5d39 to 016fdfb Compare August 15, 2025 08:20
@jessegeens jessegeens force-pushed the fix/spaces-image branch 3 times, most recently from 689e88f to 136cc4d Compare August 15, 2025 12:59
@jessegeens jessegeens requested a review from diocas August 19, 2025 09:22
@jessegeens jessegeens force-pushed the fix/spaces-image branch 2 times, most recently from 9f96e8a to bf980ab Compare August 19, 2025 13:58
* Also did some other small refactorings that were mentioned in the review
of the PR
* And added some comments
@diocas diocas merged commit 7bd7494 into master Aug 19, 2025
15 of 17 checks passed
@diocas diocas deleted the fix/spaces-image branch August 19, 2025 14:04
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.

4 participants