-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc(tutorials): publish the e2e adopter journey tutorial #472
Conversation
feat(tutorials): Initial commit to provide the tutorials structure on feature branch
Run Markdown linter and solve issues delete unused folder/files
feat(tutorials): Provide initial content and structure
chore(update): Keep feature branch up to date related to main
add CHANGELOG.md delete unused section restructure content
Adapt initial structure and content
Added generic tutorial description
(doc): Completed onboarding information material for e2e tutorial
Delete unused files/folders add NOTICE section add Admonitions
doc(tutorials): Keep feature branch up to date
add new diagram in draw.io extend the different section of the architecture view fix some linter issues
doc(tutorials): adapt architecture view
…stricting-Users doc(tutorials): Adjustments of chapter 3/Restricting users after review
doc(tutorials): Minor adjustments on chapter3/Consume Data
doc(tutorials): adapt architecture information
What's the tractusx-edc version used here? Some payloads look weird like {
"querySpec": {
"offset": 0,
"limit": 100,
"filter": "",
"range": {
"from": 0,
"to": 100
},
"criterion": ""
}
}
that looks way different than the openAPI definition. |
Its 0.5.1 the one which is also mentioned in the related standards |
What is None of these properties for a |
|
I am not sure who worked on this. Maybe @alexandrudanciu @hemantxpatel or @paullatzelsperger know the answer here. |
Hi, the correct {
"querySpec": {
"offset": 0,
"limit": 100
}
} I have referred the existing querySpec from some postman collections. Strangely it doesn't fail or warn, that's why it got skipped. |
@hemantxpatel - Link the ticket here when it's created, please. I've observed the querySpec mechanism behaving strange in a couple of similar situations. @stephanbcbauer - can you update the relevant parts in the tutorial? Ctl-F "querySpec" should show you all relevant occurences of the error above and I don't have push rights for this repo. |
I raised issue here. eclipse-tractusx/tutorial-resources#166
@arnoweiss Since these fields are unknown, they might get silently ignored during Json Serialization and should not affect any functionalities. |
Misunderstanding - sorry. I thought you'd open an issue about the EDC's behavior. I'll collect some more examples and then do it myself. Thanks for taking care of this and letting Stephan off my hook. |
To not confuse others, I will adapt the sections before we release the tutorial. |
Description
This PR is the final step to merge the feature branch to main and publish the e2e adopter journey tutorial.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: