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

MTM-62224 Create entry in the c8ydocs with gs docs #2878

Open
wants to merge 57 commits into
base: develop
Choose a base branch
from

Conversation

luja-c8y
Copy link
Contributor

@luja-c8y luja-c8y commented Feb 3, 2025

The point is to hide these docs, so only users with a link should be able to see them, that's why left navigator is hidden.
Preview: https://cumulocity-iot-c8y-docs-build-pr-2878.surge.sh/gainsight-px/

Based on: https://docs.google.com/document/d/1C7QkA99FPXq0QmexwC8z2uk7BNckxhAwqNegXZ8dIVg/edit?tab=t.0

Copy link
Contributor

github-actions bot commented Feb 3, 2025

Preview available here

@luja-c8y luja-c8y marked this pull request as ready for review February 3, 2025 12:01
@luja-c8y luja-c8y marked this pull request as draft February 3, 2025 12:01
@luja-c8y luja-c8y marked this pull request as ready for review February 5, 2025 10:12
@luja-c8y luja-c8y changed the title Add preview of docs MTM-62224 Create entry in the c8ydocs with gs docs Feb 5, 2025
@luja-c8y luja-c8y requested a review from NRusenova February 5, 2025 11:31
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
@BeateRixen
Copy link
Collaborator

@luja-c8y The screenshots are outdated and should show the latest design.

@BeateRixen BeateRixen requested a review from carlosceia February 5, 2025 13:10
@BeateRixen
Copy link
Collaborator

@carlosceia Can you please review and approve the changes in the navigation?

@luja-c8y
Copy link
Contributor Author

luja-c8y commented Feb 5, 2025

@luja-c8y The screenshots are outdated and should show the latest design.

@BeateRixen thanks, updated.

@luja-c8y luja-c8y requested a review from BeateRixen February 5, 2025 14:14
{{< /c8y-admon-info >}}

2. Configure the cookie banner for all tenants on the instance.
The cookie banner configuration operates using the same mechanism as application branding. Upload the [public-options-v2](https://drive.google.com/file/d/1pEPVTutEAziWtycklRxsalYlHUmMaNIX/view?usp=sharing), which contains the cookie banner configuration, to the {{< management-tenant >}} (Administration > Own applications). The ZIP archive comprises a JSON file that defines the cookie banner title, text, link to the privacy policy page, and the cookie description. Anonymized tracking is activated by default for all tenants and users as a component of the required cookies. If users opt out of the functional cookies, only the anonymized tracking remains enabled.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we reference this file, shouldn't it also be publicly available? https://drive.google.com/file/d/1pEPVTutEAziWtycklRxsalYlHUmMaNIX/view?usp=sharing

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I was pretty sure it was, because the original document may already have been shared with customers. I will ping Dawid as he is an owner of this one.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the end we don't want to share this, so I removed a link and suggested contacting a support to get it: 72e620f
fyi: @NRusenova @BeateRixen


All attributes were added in versions:

- 10.18.0.180

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Lukasz, it's good to keep this information as a reference for now.

content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
content/gainsight-px.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants