Skip to content

Conversation

@droslbl
Copy link
Contributor

@droslbl droslbl commented Oct 20, 2025

Greetings @wistefan

We had another ticket requesting an update on TMForum's API: https://ticketing-int.dome-marketplace.eu/#ticket/zoom/2588
In summary, update patch operation to include four attributes, since this is about including more attributes in the patch im not sure if this should be applied to the base API request or if I should include another patch operation in the extension. Please confirm this part.

Thank you in advance

@wistefan
Copy link
Collaborator

Greetings @wistefan

We had another ticket requesting an update on TMForum's API: https://ticketing-int.dome-marketplace.eu/#ticket/zoom/2588 In summary, update patch operation to include four attributes, since this is about including more attributes in the patch im not sure if this should be applied to the base API request or if I should include another patch operation in the extension. Please confirm this part.

Thank you in advance

Hi,
never change any file in the apit/tm-forum folder manually. Every change deviating from the official spec needs to go into an extension.

@droslbl
Copy link
Contributor Author

droslbl commented Oct 20, 2025

Greetings @wistefan

Got it! No changes to the original API, updated the extension API with the new params

bests

@wistefan wistefan added the minor Bigger fixes, added functionality, NO breaking changes label Oct 28, 2025
@droslbl
Copy link
Contributor Author

droslbl commented Oct 31, 2025

Hi @wistefan I just merged main into this branch, whenever is possible please merge it

@droslbl
Copy link
Contributor Author

droslbl commented Nov 4, 2025

Hi @wistefan sorry for being insistent, I cannot merge the changes even with your approval as I dont have permissions in this repository, can you please accept them in case everything is in order?
Thank you in advance

@wistefan wistefan merged commit 50aeec0 into FIWARE:main Nov 4, 2025
59 of 61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Bigger fixes, added functionality, NO breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants