Skip to content

Conversation

@PGijsbers
Copy link
Contributor

@PGijsbers PGijsbers commented Oct 5, 2025

note: There is something to be said for updating the migration further to reduce a many-to-many link table, and instead have a relevant_link table per asset type. This should be more performant. But this works and we have more important things (we can always measure what performance imapct this has, but I reckon not so much -- in any case no worse than what already was).

Change(s)

Change Type: Changed

Change Category: Interface

Changelog Entry:
Extended the maximum length of a relevant_link of AI resources from 256 characters to 2000 characters.

How to Test

Checklist

  • Tests have been added or updated to reflect the changes, or their absence is explicitly explained.
  • Documentation has been added or updated to reflect the changes, or their absence is explicitly explained.
  • A self-review has been conducted checking:
    • No unintended changes have been committed.
    • The changes in isolation seem reasonable.
    • Anything that may be odd or unintuitive is provided with a GitHub comment explaining it (but consider if this should not be a code comment or in the documentation instead).
  • All CI checks pass before pinging a reviewer, or provide an explanation if they do not.
  • The PR title matches the changelog entry's one-line description.

Related Issues

@PGijsbers PGijsbers requested a review from Taniya-Das October 5, 2025 06:40
@PGijsbers PGijsbers changed the title Extend url Extended the maximum length of a relevant_link of AI resources from 256 characters to 2000 characters. Oct 5, 2025
Copy link
Collaborator

@Taniya-Das Taniya-Das left a comment

Choose a reason for hiding this comment

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

lgtm

@PGijsbers PGijsbers merged commit 8acca8a into develop Oct 21, 2025
1 check passed
@PGijsbers PGijsbers deleted the extend-url branch October 21, 2025 02:19
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