Skip to content

Conversation

@rimi-itk
Copy link
Contributor

@rimi-itk rimi-itk commented Aug 19, 2025

https://leantime.itkdev.dk/_#/tickets/showTicket/5189

Adds support for MeMo 1.2 (cf. https://digitaliser.dk/Media/638608781984779669/MeMo%20Versionshistorik%20v1.2.pdf).

  • Updates default MeMo version to 1.2
  • Adds validation of action URLs

See https://github.com/itk-dev/serviceplatformen/blob/3173fb5f7b721145374682c145e340f0db5da4b7/docs/SF1601.md#memo-version for some details. Is using an environment variable to override the default version the way to go?

@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from 7850bdd to fe14832 Compare August 19, 2025 09:23
@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from a76a9e4 to 32afa18 Compare August 19, 2025 10:53
@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from 32afa18 to 72c9d69 Compare August 19, 2025 11:15
@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from 8afcde4 to beae623 Compare August 19, 2025 12:15
@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from beae623 to 8bf2658 Compare August 20, 2025 07:22
@rimi-itk rimi-itk force-pushed the feature/5189-memo-1.2 branch from 8bf2658 to 3173fb5 Compare August 20, 2025 07:57
@rimi-itk rimi-itk marked this pull request as ready for review August 20, 2025 08:10
@rimi-itk rimi-itk requested review from jekuaitk and martinyde August 20, 2025 08:11
Copy link
Contributor

@jekuaitk jekuaitk left a comment

Choose a reason for hiding this comment

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

Some typos, looks good otherwise!

rimi-itk and others added 4 commits August 20, 2025 10:59
Co-authored-by: Jeppe Kuhlmann Andersen <78410897+jekuaitk@users.noreply.github.com>
Co-authored-by: Jeppe Kuhlmann Andersen <78410897+jekuaitk@users.noreply.github.com>
@rimi-itk rimi-itk requested a review from jekuaitk August 20, 2025 09:03
@rimi-itk rimi-itk merged commit 447f1f4 into main Aug 20, 2025
6 checks passed
@rimi-itk rimi-itk deleted the feature/5189-memo-1.2 branch August 20, 2025 09:16
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