Skip to content

Update supported objects for Airdrop #250

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 4 commits into from
Jun 5, 2025
Merged

Conversation

arturo-aparicio
Copy link
Contributor

@arturo-aparicio arturo-aparicio commented Jun 5, 2025

Work: https://app.devrev.ai/devrev/works/ISS-180330

Update the supported objects for Airdrop

@arturo-aparicio arturo-aparicio requested a review from a team as a code owner June 5, 2025 21:50
Copy link
Contributor

github-actions bot commented Jun 5, 2025

🌟 EkLine Reviewer

Hello! I’m here to help improve your docs. I’ve reviewed your pull request, and left in-line suggestions for quick fixes. For details, visit the Analytics Page.

For questions or feedback, please email support@ekline.io.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

EkLine

🚫 [EkLine]

'conversation' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Consider using 'customer' instead of 'rev'. (EK25033)

| `participant_uids` | reference (collection) -> [#category:user] | | Globally unique ID of DevRev rev users |


🚫 [EkLine]

'custom_link' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'custom_part' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'devu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'directory' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'dm' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'enhancement' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'feature' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'group' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'incident' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'issue' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)

| `release_version_ids` | reference (collection) -> [#record:version] | | Versions that will contain the resolving commit IDs |


⚠️ [EkLine]

Consider using 'customer' instead of 'Rev'. (EK25033)

| `rev_org_ids` | reference (collection) -> [#record:account #record:revo] | | Rev orgs associated to issues |


🚫 [EkLine]

'link' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'linkable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'meeting' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'microservice' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'object_member' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'opportunity' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'product' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'revu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'runnable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'sysu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'tag' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'task' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'ticket' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Consider using 'workspace' instead of 'rev org'. (EK25033)

| `rev_oid` | reference -> [#record:account #record:revo] | | Globally unique ID for a DevRev rev org |

Copy link
Contributor

github-actions bot commented Jun 5, 2025

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

EkLine

🚫 [EkLine]

'incident' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'issue' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)

| `release_version_ids` | reference (collection) -> [#record:version] | | Versions that will contain the resolving commit IDs |


⚠️ [EkLine]

Consider using 'customer' instead of 'Rev'. (EK25033)

| `rev_org_ids` | reference (collection) -> [#record:account #record:revo] | | Rev orgs associated to issues |


🚫 [EkLine]

'link' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'linkable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'meeting' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'microservice' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'object_member' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'opportunity' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'product' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'revu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'runnable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'sysu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'tag' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'task' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'ticket' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Consider using 'workspace' instead of 'rev org'. (EK25033)

| `rev_oid` | reference -> [#record:account #record:revo] | | Globally unique ID for a DevRev rev org |

Copy link
Contributor

github-actions bot commented Jun 5, 2025

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

EkLine

🚫 [EkLine]

'conversation' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Consider using 'customer' instead of 'rev'. (EK25033)

| `participant_uids` | reference (collection) -> [#category:user] | | Globally unique ID of DevRev rev users |


🚫 [EkLine]

'custom_link' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'custom_part' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'devu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'directory' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'dm' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'enhancement' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'feature' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'group' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'incident' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'issue' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)

| `release_version_ids` | reference (collection) -> [#record:version] | | Versions that will contain the resolving commit IDs |


⚠️ [EkLine]

Consider using 'customer' instead of 'Rev'. (EK25033)

| `rev_org_ids` | reference (collection) -> [#record:account #record:revo] | | Rev orgs associated to issues |


🚫 [EkLine]

'link' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'linkable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'meeting' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'microservice' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'object_member' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'opportunity' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'product' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'revu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'runnable' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'sysu' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'tag' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'task' should use sentence-style capitalization. (EK00037)


🚫 [EkLine]

'ticket' should use sentence-style capitalization. (EK00037)


⚠️ [EkLine]

Consider using 'workspace' instead of 'rev org'. (EK25033)

| `rev_oid` | reference -> [#record:account #record:revo] | | Globally unique ID for a DevRev rev org |

[ ▲ top](#summary)

---

## account
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'account' should use sentence-style capitalization. (EK00037)

| `ACTIVE` | ACTIVE | - |
| `INACTIVE` | INACTIVE | - |
| `LOCKED` | LOCKED | - |

### Structs

#### postal_address
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'postal_address' should use sentence-style capitalization. (EK00037)

|-----|------|-------------|
| `dev_users` | Dev users | "Dynamic dev users group created by DevRev" |
| `none` | No group | "No group selected" |
| `rev_users` | Rev users | "Dynamic rev users group created by DevRev" |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Suggestions:

  • Consider using 'customer' instead of 'Rev'. (EK25033)
  • Consider using 'customer' instead of 'rev'. (EK25033)

Suggested change
| `rev_users` | Rev users | "Dynamic rev users group created by DevRev" |
| `rev_users` | Customer users | "Dynamic customer users group created by DevRev" |

| `customer_admins` | - | - |
| `customers` | - | - |
| `verified_customers` | - | - |

### Structs

#### resource
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'resource' should use sentence-style capitalization. (EK00037)

@@ -243,6 +378,19 @@ This document contains the list of all DevRev object types that are supported fo
| `name` | text | | The name of the channel. If provided, this will be unique across all channels in the Dev org. It's case insensitive and must contain the characters [A-Za-z0-9_-]. |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Suggestions:

  • Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
  • Consider using 'TBD' instead of 'Dev org'. (EK25033)

Suggested change
| `name` | text | | The name of the channel. If provided, this will be unique across all channels in the Dev org. It's case insensitive and must contain the characters [A-Za-z0-9_-]. |
| `name` | text | | The name of the channel. If provided, this is unique across all channels in the Dev org. It's case insensitive and must contain the characters [A-Za-z0-9_-]. |

| `qualification` | Qualification | - |
| `stalled` | Stalled | - |
| `validation` | Validation | - |

### Structs

#### money
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'money' should use sentence-style capitalization. (EK00037)

@@ -937,15 +1415,12 @@ This document contains the list of all DevRev object types that are supported fo
| `description` | rich_text | | Description of the Rev user. |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Consider using 'customer' instead of 'Rev user'. (EK25033)

Suggested change
| `description` | rich_text | | Description of the Rev user. |
| `description` | rich_text | | Description of the customer. |

@@ -937,15 +1415,12 @@ This document contains the list of all DevRev object types that are supported fo
| `description` | rich_text | | Description of the Rev user. |
| `display_handle` | text | | The user's display handle. This field is deprecated now. Use display_name field instead. |
| `display_name` | text | ✔︎ | The user's display name. The name is non-unique and mutable. |
| `display_picture` | reference -> [#record:artifact] | | Artifact ID of the user's display picture |
| `email` | text | | Email address of the user. |
| `external_uid` | text | | External ref is a mutable unique identifier for a user within the Rev organization from your primary customer record. If none is available, a good alternative is the email address/phone number which could uniquely identify the user. If none is specified, a system-generated identifier will be assigned to the user. |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Suggestions:

  • Where possible, do not structure sentences in future tense. Use present tense instead. (EK00005)
  • Consider using 'customer' instead of 'Rev'. (EK25033)

Suggested change
| `external_uid` | text | | External ref is a mutable unique identifier for a user within the Rev organization from your primary customer record. If none is available, a good alternative is the email address/phone number which could uniquely identify the user. If none is specified, a system-generated identifier will be assigned to the user. |
| `external_uid` | text | | External ref is a mutable unique identifier for a user within the customer organization from your primary customer record. If none is available, a good alternative is the email address/phone number which could uniquely identify the user. If none is specified, a system-generated identifier is assigned to the user. |

@@ -937,15 +1415,12 @@ This document contains the list of all DevRev object types that are supported fo
| `description` | rich_text | | Description of the Rev user. |
| `display_handle` | text | | The user's display handle. This field is deprecated now. Use display_name field instead. |
| `display_name` | text | ✔︎ | The user's display name. The name is non-unique and mutable. |
| `display_picture` | reference -> [#record:artifact] | | Artifact ID of the user's display picture |
| `email` | text | | Email address of the user. |
| `external_uid` | text | | External ref is a mutable unique identifier for a user within the Rev organization from your primary customer record. If none is available, a good alternative is the email address/phone number which could uniquely identify the user. If none is specified, a system-generated identifier will be assigned to the user. |
| `full_name` | text | | Full name of the user. |
| `phone_numbers` | text (collection) | | Phone numbers of the user. |
| `rev_org` | reference -> [#record:account #record:revo] | | The ID of the parent Rev Organization. |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Consider using 'customer' instead of 'Rev'. (EK25033)

Suggested change
| `rev_org` | reference -> [#record:account #record:revo] | | The ID of the parent Rev Organization. |
| `rev_org` | reference -> [#record:account #record:revo] | | The ID of the parent customer Organization. |

@@ -1141,7 +1639,46 @@ This document contains the list of all DevRev object types that are supported fo
| `title` | text | ✔︎ | Title of the work object |
| `visibility` | enum | | The visibility of the object. If 'internal', the object is visible within the Dev organization. If 'external', it is visible to both the Dev organization and Rev users. If not set, the default visibility is 'external'. |
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Consider using 'customer' instead of 'Rev'. (EK25033)

Suggested change
| `visibility` | enum | | The visibility of the object. If 'internal', the object is visible within the Dev organization. If 'external', it is visible to both the Dev organization and Rev users. If not set, the default visibility is 'external'. |
| `visibility` | enum | | The visibility of the object. If 'internal', the object is visible within the Dev organization. If 'external', it is visible to both the Dev organization and customer users. If not set, the default visibility is 'external'. |

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

Avoid using the term 'Dev user'. (EK25015)

| `owned_by` | reference (collection) -> [#category:user] | ✔︎ | List of Dev user IDs owning this Account. |

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'airdrop_authorization_policy' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'airdrop_platform_group' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'article' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [EkLine]

Consider using 'customer' instead of 'Rev user'. (EK25033)

| `shared_with` | permission (collection) | | The list of Rev user, groups and dynamic groups with whom the article is shared and the corresponding roles. |

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

Use 'DevRev' instead of 'devrev'. (EK25050)

| `content_blocks` | reference | | Content blocks used in the article (relevant only for devrev/rt articles). |

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'capability' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'channel' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'comment' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [EkLine]

'component' should use sentence-style capitalization. (EK00037)

Copy link
Contributor

github-actions bot commented Jun 5, 2025

Copy link
Contributor

@bc-devrev bc-devrev left a comment

Choose a reason for hiding this comment

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

A few lines that don't seem to be needed but I'll let you decide.

@@ -1,13 +1,14 @@
# Supported DevRev object types for Airdrop

Last updated: _2025-04_
Last updated: _June 5, 2025_
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this need to be exposed to the user?

Copy link
Contributor

github-actions bot commented Jun 5, 2025

@arturo-aparicio arturo-aparicio merged commit b8af6b6 into main Jun 5, 2025
5 of 6 checks passed
@arturo-aparicio arturo-aparicio deleted the art/update-metadata branch June 5, 2025 23:41
tilenkavcic pushed a commit that referenced this pull request Jun 9, 2025
* Update supported objects for Airdrop

* Update supported-object-types.mdx

* Update supported-object-types.mdx

* Update supported-object-types.mdx
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