Skip to content

Update template #8

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
Jul 2, 2025
Merged

Conversation

dusano
Copy link
Contributor

@dusano dusano commented Jul 2, 2025

Update to the snap-in template and some additional documentation. Here are the details of this update:

  1. The reference to docs/airdrop_sdk_library_documentation.md was moved to Definitions sections so that it's available also to the conformance tests.

  2. In devrev-snapin-template.plain we've added a small implementation hint that AI consistently got wrong.

  3. In docs/airdrop_sdk_library_documentation.md 'function_name' and 'event_type' attributes were missing in AirdropEvent object. This was really confusing AI so I added it.

  4. We are also adding docs/mapping-reasons.mdx that is needed for initial domain mapping and will be referenced in specific snap-ins (e.g. Trello). The file was copied from https://github.com/devrev/fern-api-docs/blob/main/fern/docs/pages/airdrop/data-model/mapping-reasons.mdx

  5. From runner.ts I've deleted some unused imports.

Work item: https://app.devrev.ai/devrev/works/ISS-188589

Copy link
Contributor

@patricijabrecko patricijabrecko left a comment

Choose a reason for hiding this comment

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

LG

@patricijabrecko patricijabrecko merged commit a44341e into devrev:main Jul 2, 2025
2 checks passed
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