-
Notifications
You must be signed in to change notification settings - Fork 35
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
Show V3 focused landscape of extensions #952
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Jakub Balhar <[email protected]>
Signed-off-by: Jakub Balhar <[email protected]>
Signed-off-by: Jakub Balhar <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approval for code changes. The content looks good to me too, but might want another set of eyes on it.
@MarkAckert I want to discuss on Onboarding squad meeting, as the full included landscape works properly while the excerpts doesn't allow clicking on them. |
This is a great first draft Jakub! Some suggestions:
Note: I am not advocating that all section headers be capitalized - I used that format in this comment to highlight those specific headers. I think you can leave them as is. |
Signed-off-by: Jakub Balhar <[email protected]>
I reflect the first three points fully. As for the fourth point, the View the Full Interactive landscape is part of the page integrated from Open Mainframe Project and as such I can't do much about it. What I would like to discuss is integrating the view for the first and second parts the same way as for the third one as there is an issue with the cards in the Open Mainframe Project's page. |
Signed-off-by: Jakub Balhar <[email protected]>
No description provided.