Skip to content

Remove inactive destinations #2827

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

Merged
merged 2 commits into from
May 2, 2022
Merged

Remove inactive destinations #2827

merged 2 commits into from
May 2, 2022

Conversation

jinapark202
Copy link
Contributor

JIRA

This PR removes Seg and goedle.io from the docs as the companies no longer exists. The destinations have been marked as Unavailable.

Merge timing

ASAP once approved

Related issues (optional)

@jinapark202 jinapark202 requested review from stayseesong, markzegarelli and a team as code owners May 2, 2022 19:58
@markzegarelli markzegarelli merged commit 4737b75 into develop May 2, 2022
@markzegarelli markzegarelli deleted the remove-seg branch May 2, 2022 20:26
@github-actions
Copy link
Contributor

github-actions bot commented May 2, 2022

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

markzegarelli pushed a commit that referenced this pull request May 3, 2022
* Added Criteo Audiences documentation

* updated Criteo description

* edits

* new edits to text

* edits to text

* edits to text

* edits

* reformatted

* updated description

* updated description

* added include statement to pull available actions, fields, settings

* Formatted warning

* added a note about delay

* update word accept list

* criteo doc edits

* Apply suggestions from code review

Co-authored-by: forstisabella <[email protected]>

* Draft of GA4 changes

* Update src/connections/destinations/catalog/actions-criteo-audiences/index.md

Co-authored-by: forstisabella <[email protected]>

* Documentation for the obfuscate feature and how to use it

* updates to ga4 draft

* Apply suggestions from code review

* note on nesting

* small update to gtag docs (#2820)

* [netlify-build]

* [netlify-build]

* Apply suggestions from code review

* Clarify which actions are obfuscated

* Update src/connections/destinations/catalog/actions-google-analytics-4/index.md

* Update src/connections/destinations/catalog/actions-google-analytics-4/index.md

* Update src/connections/destinations/catalog/actions-google-analytics-4/index.md

* Update src/connections/destinations/catalog/actions-google-analytics-4/index.md

* Update src/connections/destinations/catalog/actions-google-analytics-4/index.md

* Apply suggestions from code review

Co-authored-by: rchinn-segment <[email protected]>

* Remove inactive destinations (#2827)

* remove inactive destinations

* ran make catalog

Co-authored-by: markzegarelli <[email protected]>

* Apply suggestions from code review

Co-authored-by: stayseesong <[email protected]>

* Apply suggestions from code review

Co-authored-by: forstisabella <[email protected]>

* Update index.md

* Update proxy process

* Proper merge of upstream

* edits

* Fixed some broken URLs (#2829)

* Revert "Bundle Obfuscation [DOC-508]"

* Update index.md

Co-authored-by: Nadine Malek <[email protected]>
Co-authored-by: Sai Harika Punyamurthula <[email protected]>
Co-authored-by: Sai Harika Punyamurthula <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: stayseesong <[email protected]>
Co-authored-by: forstisabella <[email protected]>
Co-authored-by: kdaswani <[email protected]>
Co-authored-by: Ariel Silvestri <[email protected]>
Co-authored-by: Segment Docs Robot <[email protected]>
Co-authored-by: rchinn-segment <[email protected]>
Co-authored-by: jinapark202 <[email protected]>
Co-authored-by: Niall Brennan <[email protected]>
Co-authored-by: pwseg <[email protected]>
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.

2 participants