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

[Fix][Doc] Fix klaviyo doc 404 link #7969

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

YOMO-Lee
Copy link
Contributor

@YOMO-Lee YOMO-Lee commented Nov 3, 2024

At present, the reference link provided by Klaviyo connector is 404 inaccessible and needs to be modified, otherwise it will hinder the operation of ST project CI

At present, the reference link provided by Klaviyo connector is 404 inaccessible and needs to be modified, otherwise it will hinder the operation of ST project CI
@liunaijie
Copy link
Member

Please update to right related link.

@@ -45,7 +45,7 @@ http request url

API private key for login, you can get more detail at this link:

https://developers.klaviyo.com/en/docs/retrieve_api_credentials
https://developers.klaviyo.com/en/docs
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
https://developers.klaviyo.com/en/docs
https://developers.klaviyo.com/en/docs/authenticate_#private-key-authentication

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok,done
please review again

Change the connection to the correct address
@YOMO-Lee
Copy link
Contributor Author

YOMO-Lee commented Nov 4, 2024

Please update to right related link.

ok,done
please review again

@Hisoka-X Hisoka-X changed the title [Bug] [Klaviyo DOC] Fix 404 link [Fix][Doc] Fix klaviyo doc 404 link Nov 4, 2024
@hailin0 hailin0 merged commit 019af39 into apache:dev Nov 4, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants