Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mara3l
Copy link
Contributor

@Mara3l Mara3l commented Aug 9, 2023

No description provided.

@ghost
Copy link

ghost commented Aug 9, 2023

Build succeeded (check pipeline).

{{- template "_internal/opengraph.html" . -}}
{{- template "_internal/google_news.html" . -}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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.

Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

@Mara3l Mara3l Aug 9, 2023

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 */}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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>
Copy link
Contributor Author

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"}}
Copy link
Contributor Author

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 */}}
Copy link
Contributor Author

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

@Mara3l
Copy link
Contributor Author

Mara3l commented Aug 9, 2023

Please ping me when you are nearly done with this, as we would need to also update all the scss :)

@Mara3l Mara3l changed the title .. DO NOT MERGE: just a diff between python and this repo Aug 21, 2023
@Mara3l Mara3l added the do not merge Do not merge this yet label Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant