-
Notifications
You must be signed in to change notification settings - Fork 3
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
DO NOT MERGE: just a diff between python and this repo #26
base: master
Are you sure you want to change the base?
Conversation
Build succeeded (check pipeline). |
{{- template "_internal/opengraph.html" . -}} | ||
{{- template "_internal/google_news.html" . -}} |
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.
Incompatible with newer Hugo
@@ -1,4 +1,3 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
Let's look at this separately. Keep as is for now.
@@ -1,10 +1,4 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
How about search for GD.UI?
@@ -1,14 +1,92 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
@czechian , please look at this and try to merge it.
@@ -0,0 +1,18 @@ | |||
{{/* PETR: Definition of the superheading tag */}} |
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.
Rename this and remove the hardcoded color, please
@@ -1,6 +1,6 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
Please split this into two partials and create a switch for these two.
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.
Or we can have two latest versions (include dev in latest) and have a new devVersion partial or something like that...
@@ -1,6 +1,4 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
This is just an old version - please update from GD.UI and merge it together
@@ -1,13 +0,0 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
This is just design update - Dan didn't want to have red font
@@ -1,4 +1,3 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
md5 fix
@@ -1,22 +1,17 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
Use the one from docs theme, as it fixes nested shortcodes
@@ -1,7 +1,10 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
We can overwrite this in the PySDK
@@ -1,18 +1,13 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
Can be overwritten in PySDK
@@ -0,0 +1,5 @@ | |||
<tr> |
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.
Can be in PySDK
@@ -0,0 +1,20 @@ | |||
{{ $partitle := .Get "title"}} |
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.
Can be in PySDK
@@ -1,4 +1,3 @@ | |||
{{/* (C) 2023 GoodData Corporation */}} |
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.
Keep the newer version, fixes md-5
Please ping me when you are nearly done with this, as we would need to also update all the scss :) |
No description provided.