Skip to content

Docs/terraform provider#2874

Open
eldadfux wants to merge 28 commits intomainfrom
docs/terraform-provider
Open

Docs/terraform provider#2874
eldadfux wants to merge 28 commits intomainfrom
docs/terraform-provider

Conversation

@eldadfux
Copy link
Copy Markdown
Member

@eldadfux eldadfux commented Apr 9, 2026

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

eldadfux added 21 commits April 8, 2026 16:50
Add overview with Terraform basics, self-hosting redirect note, and
guides for provider configuration, database resources, and data sources.
Link to appwrite/terraform-provider-appwrite; align content with README.

Made-with: Cursor
… nav

Align docs with provider: bucket, messaging_provider, messaging_topic.
Move database guide under resources/databases; add resources/storage and
resources/messaging. Nav: Getting started, Resources, Data sources.
Redirect old /terraform/databases path.

Made-with: Cursor
Add terraform.svg to icon pipeline (svgtofont + sprite), use web-icon-terraform
in docs Tooling menu. Colors normalized by optimizer; sidebar uses currentColor
like other nav icons.

Made-with: Cursor
…th registry docs

Add registry URLs for provider index, generated docs, and per-resource pages.
Pin required_providers example to ~> 0.0.4; cross-link GitHub for source.

Made-with: Cursor
Add 2026-04-08 changelog on IaC value, docs, registry, and catalog.
Add integrations/terraform-provider page under Deployments with cover and avatar SVGs.

Made-with: Cursor
Product announcement with docs, registry, and integrations links; cover SVG.

Made-with: Cursor
…weaks

Add introducing-terraform-provider-for-appwrite post and levi-van-noort author.
Use ASCII hyphens in Terraform provider and databases docs.
Include icon font output assets and avatar.

Made-with: Cursor
These copies are not produced by the icon build; drop them from the repo.

Made-with: Cursor
Prevents accidental commits of Finder "file 2.ext" duplicates.

Made-with: Cursor
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 9, 2026

Greptile Summary

This PR adds full documentation for the official appwrite/appwrite Terraform provider: a new docs section under /docs/tooling/terraform (overview, provider configuration, and eight resource guides), an integrations catalog entry, a blog post, and a changelog entry. Previously flagged concerns are resolved — the mongo icon is present in both types.ts and sprite.svelte, the blog post changelog link correctly targets /changelog/entry/2026-04-19 matching the renamed file, and code.ts now has a comment explaining the HCL→ini fallback.

Confidence Score: 5/5

Safe to merge — no P0/P1 issues found and all previously flagged concerns are resolved.

All prior review concerns have been addressed: changelog link matches the renamed file, mongo icon is intact, and the HCL alias comment is in place. No new logic, routing, or icon regressions were found. Remaining feedback from earlier rounds was P2-level and is not repeated here.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/blog/post/introducing-terraform-provider-for-appwrite/+page.markdoc New blog post introducing the Terraform provider; changelog link correctly points to /changelog/entry/2026-04-19, matching the renamed changelog file
src/routes/changelog/(entries)/2026-04-19.markdoc Changelog entry for Terraform provider; filename, slug, date, and blog post link are all consistent
src/lib/components/ui/icon/types.ts Adds 'terraform' to IconType union; 'mongo' remains present at line 15 and the corresponding sprite symbol is untouched
src/lib/utils/code.ts Registers hcl/terraform/tf as aliases for ini; comment added explaining why no native HCL grammar is used
src/redirects.json Adds a single redirect from the old /docs/tooling/terraform/databases path to /docs/tooling/terraform/resources/databases; no broken redirect chains present
src/routes/docs/tooling/terraform/+page.markdoc New Terraform provider overview page; resource table and next-steps cards all link to valid sub-pages
src/routes/docs/tooling/terraform/provider/+page.markdoc Provider configuration guide covering Cloud, Community Edition, environment variables, and project scoping; content is accurate and complete
src/routes/docs/tooling/terraform/resources/messaging/+page.markdoc Messaging resources guide with examples for Sendgrid, SMTP, Twilio, and FCM; FCM file() usage present without a .gitignore note (previously flagged)
src/routes/docs/tooling/terraform/+layout.svelte Sidebar navigation for Terraform docs section; all hrefs point to files that exist in this PR
src/routes/integrations/terraform-provider/+page.markdoc Integration catalog entry for Terraform provider; cover image path and linked docs all resolve correctly
src/routes/docs/Sidebar.svelte Adds Terraform provider entry to the tooling section of the docs sidebar with correct href and web-icon-terraform icon
src/routes/docs/tooling/terraform/resources/databases/+page.markdoc TablesDB resource guide with full example covering database, tables, columns of various types, and index creation

Reviews (6): Last reviewed commit: "docs(terraform): refresh blog rewrite, c..." | Re-trigger Greptile

Comment thread src/lib/components/ui/icon/types.ts Outdated
Comment on lines +60 to +64
}
```

## Topics {% #topics %}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Missing warning about committing service account JSON to git

The FCM example reads credentials with file("firebase-service-account.json") but there is no note warning users to add that file to .gitignore and never commit it. Readers following this example for the first time may inadvertently leak their service account key.

Consider adding a short note here:

Important: Add firebase-service-account.json to your .gitignore. Never commit secret files to version control; use environment variables or a secrets manager for CI.

Comment thread src/lib/utils/code.ts
Resolve conflicts: icon sprite/types (terraform + restored mongo), changelog
split for 2026-04-08 entries, github-stars, and regenerated icon output.

Made-with: Cursor
Comment thread src/routes/blog/post/introducing-terraform-provider-for-appwrite/+page.markdoc Outdated
…s, assets

Align TablesDB resource names and capabilities; add Auth, Functions, Sites,
Webhooks, and Backups pages; fold data source into Databases; update redirects.
Refresh Terraform blog, changelog, and integration copy; new covers and avatar.

Made-with: Cursor
Comment thread src/routes/blog/post/introducing-terraform-provider-for-appwrite/+page.markdoc Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this blog is still missing some resources, other than that it looks fine. I did a rewrite of it here: https://github.com/appwrite/website/pull/2887/changes#diff-77917839cc715ba8a72fb8737c10434dfdfd13fd29086b3f7d9837fb26deca5c

---
layout: changelog
title: "Terraform provider for Appwrite"
date: 2026-04-17
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
date: 2026-04-17
date: 2026-04-19

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.

3 participants