Skip to content

Test assignment from Katkov Ivan - #8

Open
Pseudolukian wants to merge 2 commits into
adaptyteam:mainfrom
Pseudolukian:pseudolukian/test-assignment
Open

Test assignment from Katkov Ivan#8
Pseudolukian wants to merge 2 commits into
adaptyteam:mainfrom
Pseudolukian:pseudolukian/test-assignment

Conversation

@Pseudolukian

Copy link
Copy Markdown

1. Strategy on improving Adapty documentation

I will divide the work on improving the documentation into two main blocks: content work and DocOps work. Content work on the documentation to enhance user experience:

  • First, I will thoroughly study the Adapty documentation from start to finish. I’ll go through the key instructions as a regular user and test everything myself. If I find any confusing or unclear sections, I’ll fix them.
  • I’ll analyze competitors’ documentation: RevenueCat, Qonversion, Apphud, Purchasely, and compare them with Adapty’s documentation. I’ll identify the strengths of each documentation, noting the most successful practices and approaches to apply them to Adapty’s documentation.
  • I’ll conduct an SEO analysis of the existing Adapty documentation and competitors’ documentation using SEMrush, highlighting the most effective SEO patterns.
  • I’ll develop an editorial policy through which I’ll apply the well-performing approaches and SEO insights found from competitors.
  • I’ll review real user feedback: Adapty Feedback.

DocOps work to improve the documentation:

  • Transition from a flat structure to a tree-like structure that logically mirrors the documentation’s TOC.
  • Use #include and {{ var }} for content reuse, reducing errors, and ensuring greater consistency in the documentation.

2. Write documentation on the Overview feature

I rewrote the documentation in versioned_docs/version-3.0/overview.md, aiming to add more clarity and detail.

3. Identify issues with the SDK installation flow and provide an improved version

I worked a bit on versioned_docs/version-3.0/installation-of-adapty-sdks.md. Unfortunately, my laptop doesn’t allow me to fully install Dart, Swift, Flutter, and Android Studio to test the documentation comprehensively, so I did what I could for this task.

4. Identify issues with the Server Side API and (optionally, as a bonus) provide an improved version

Here, I already have some observations:

  • Firstly, the documentation doesn’t clearly explain when to use api-secret-key versus public-api-key.
  • Secondly, for some reason, the Postman section is nested inside the Requests section.
  • Thirdly, there are duplicated 200 and 40X responses across the Requests pages.
  • Fourthly, it’s generally unclear why this section is useful or how it can be applied. It lacks practical examples and an explanation of its value.

Overall, my impression of the documentation is mostly positive. It’s clear that the documentation is still young, and the value of many sections hasn’t been fully established yet. I hope I can be helpful, and that my experience and approach will contribute to making Adapty’s documentation even better.

@eandreeva-twr
eandreeva-twr self-requested a review as a code owner November 24, 2025 13:28
eandreeva-twr added a commit that referenced this pull request Aug 11, 2026
* Document Ads Manager commands in the Developer CLI

Add two pages for the `adapty asa` command topic from adapty-cli PR #8:

- developer-cli-ads-manager: task-oriented guide covering prerequisites,
  bulk keyword operations, metrics in scripts, automation rules, and
  idempotent re-runs
- developer-cli-ads-manager-reference: all 33 commands with arguments,
  flags, and accepted values, nested under the guide in api.json

Written from the command source rather than the PR README, which is
wrong on --period-unit casing, omits five campaigns create flags, and
misses the connect --timeout flag.

Retitle the existing reference to "Command reference" (slug unchanged)
and narrow its scope statement. Cross-link the two surfaces from
Ads Manager articles for keywords, campaigns, automations, and metrics.

* Add competitors command, drop CLI skill callouts, reframe for agents

- Document `asa competitors summary` from adapty-cli commit aedd248:
  new Competitors section in the reference, a guide section, and a
  cross-link from Market Intelligence. It is a subset of the dashboard
  feature, so both sections send the reader back for filtering and
  add-to-campaign.
- Remove the "Using an AI assistant?" skill callout from all four
  Developer CLI articles.
- Split "Before you start" into Prerequisites and Differences from the
  rest of the CLI.
- Lead with agentic workflows rather than scripts, since that is the
  primary use case for reading metrics from the CLI.
- Link the CLI from the Ads Manager hub, and finish its truncated
  "Get started" sentence.

* Verify against CLI 0.4.0 and document the ads-manager skill

Verification against the shipped package rather than the PR README
corrected four things:

- Ads Manager has its own pagination flags: default 100 / max 1000,
  not the Developer API's 20 / 100.
- `product-pages sync` success semantics were stated backwards.
- `orgs list` and `apps list` each return two identifiers, and the
  flags accept only one — passing `org_id` or `adam_id` where a UUID
  is expected silently matches nothing.
- The list-filter matrix said "all lists", which wrongly included
  apps, orgs and automations. Now enumerated.

Also documents behavior absent from the PR source: metrics date-window
caps by grouping, the analytics budget, the automatic retry on a short
Retry-After, and that a top-N question is one ordered call.

Adds a page for the ads-manager AI skill with a reusable install
snippet, linked from the CLI guide, the Ads Manager hub, and the
AI Agent article, which it contrasts with (advisory versus acting).

* Note the *.adapty.io domain allowlist requirement for Cowork

Corporate Cowork workspaces restrict outbound domains, so the skill
cannot reach Adapty until an administrator allowlists *.adapty.io.
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