Skip to content

docs(slack): remove archival reference from Download files#4867

Merged
waleedlatif1 merged 1 commit into
stagingfrom
waleedlatif1/slack-docs-no-archival
Jun 3, 2026
Merged

docs(slack): remove archival reference from Download files#4867
waleedlatif1 merged 1 commit into
stagingfrom
waleedlatif1/slack-docs-no-archival

Conversation

@waleedlatif1
Copy link
Copy Markdown
Collaborator

Summary

Slack Marketplace approved the app on the condition that the docs not reference archiving/backing up Slack data (per https://docs.slack.dev/slack-marketplace/slack-marketplace-app-guidelines-and-requirements/#suitable). The reviewer flagged the "Download files" line.

  • Change "Retrieve files shared in Slack channels for processing or archival" → "for processing within a workflow" in the Slack tool docs.
  • Applied to en (flagged) plus the de/es/fr translations that mirror the same phrase.

Other is_archived / excludeArchived / slack.com/archives/... references are legitimate API fields/params and are left as-is.

Type of Change

  • Docs

Testing

Manual — verified no remaining archival use-case language in the Slack docs.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jun 3, 2026 6:46pm

Request Review

@cursor
Copy link
Copy Markdown

cursor Bot commented Jun 3, 2026

PR Summary

Low Risk
Documentation-only copy edits with no runtime, security, or data-handling impact.

Overview
Updates Slack tool documentation so Download files no longer describes backing up or archiving Slack data, aligning with Slack Marketplace app guidelines.

In en, de, es, and fr slack.mdx intro bullets, the phrase is reframed from retrieval for processing or archival to retrieval for processing within a workflow. No product code or API behavior changes.

Reviewed by Cursor Bugbot for commit 67f947c. Configure here.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 3, 2026

Greptile Summary

Removes the word "archival" from the "Download files" capability description in the Slack tool docs to comply with Slack Marketplace app guidelines. The change is applied consistently across all four locale files (en, de, es, fr).

  • en: "for processing or archival" → "for processing within a workflow"
  • de/es/fr: Equivalent archival-reference phrases replaced with their respective locale's phrasing for "within a workflow"

Confidence Score: 5/5

This is a documentation-only change with no logic or code modifications — safe to merge.

Each changed file has exactly one line updated, the wording change is consistent and accurate across all four locales, and no code, configuration, or API behavior is touched.

No files require special attention.

Important Files Changed

Filename Overview
apps/docs/content/docs/en/tools/slack.mdx Single-line doc change replacing "for processing or archival" with "for processing within a workflow" to satisfy Slack Marketplace requirements.
apps/docs/content/docs/de/tools/slack.mdx German translation updated: "zur Verarbeitung oder Archivierung ab" → "zur Verarbeitung in einem Workflow ab", consistent with the English change.
apps/docs/content/docs/es/tools/slack.mdx Spanish translation updated: "para procesamiento o archivo" → "para procesamiento en un flujo de trabajo", consistent with the English change.
apps/docs/content/docs/fr/tools/slack.mdx French translation updated: "pour traitement ou archivage" → "pour traitement dans un workflow", consistent with the English change.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Slack Marketplace Review] -->|Flags archival language| B[PR: Update Download files description]
    B --> C[en/tools/slack.mdx]
    B --> D[de/tools/slack.mdx]
    B --> E[es/tools/slack.mdx]
    B --> F[fr/tools/slack.mdx]
    C -->|processing or archival| G[processing within a workflow]
    D -->|Verarbeitung oder Archivierung| H[Verarbeitung in einem Workflow]
    E -->|procesamiento o archivo| I[procesamiento en un flujo de trabajo]
    F -->|traitement ou archivage| J[traitement dans un workflow]
Loading

Reviews (1): Last reviewed commit: "docs(slack): remove archival reference f..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit b98fbfe into staging Jun 3, 2026
10 checks passed
@waleedlatif1 waleedlatif1 deleted the waleedlatif1/slack-docs-no-archival branch June 3, 2026 18:46
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.

1 participant