Skip to content
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

docs: update help text for "accept limit" for schema app creation #579

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rossiam
Copy link
Collaborator

@rossiam rossiam commented Jan 5, 2024

Updated help text for the acceptLimit field in schema invitation creation to match new limit and new information.

Copy link

changeset-bot bot commented Jan 5, 2024

🦋 Changeset detected

Latest commit: 15f1147

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smartthings/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

helpText: 'Enter the maximum number of installs you want to allow for the invitation.\n' +
'This limits the number of installs but we use the "accept" terminology here to match the API call.\n' +
'There is also a limit of 30 days that cannot be changed. After that you will need to create a new invitation.\n' +
'Leave blank for a maximum of 500.',
Copy link
Contributor

@Sitlintac Sitlintac Jan 10, 2024

Choose a reason for hiding this comment

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

Here's a suggestion for alternate wording:

Invitations are limited to a maximum of 500 installations (called "accept"s by the API) and are only valid for 30 days.
Once either the install limit or the time limit is reached, you will need to create a new invitation.
Leave blank to accept the maximum install limit or enter a smaller cap if desired.

The format of this suggested wording may be too different from our other help text to be used. And I'm not entirely sure it's better than what you already have. But I think having so much other information between "Enter the..." and "Leave blank for..." has the potential to be confusing.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I double-checked and 500 is just the default, not the absolute maximum. How about this text?

Invitations are limited to a maximum of 500 installations by default (called "accept"s by the API) and are only valid for 30 days.
Once either the install limit or the time limit is reached, you will need to create a new invitation.
Leave blank to accept the default or enter a different install limit if desired.

Copy link
Contributor

Choose a reason for hiding this comment

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

If the API doesn't mention the word "installations" anywhere, would it be beneficial to forgo using "installations" in the CLI help text to avoid introducing a new term?

Invitations are limited to a maximum of 500 acceptances by default and are valid for 30 days.
Once either the accept limit or the time limit is reached, you will need to create a new invitation.
Leave blank to accept the default limit of 500, or enter a different acceptance limit if desired.

I do think the word "installations" is more accurate here; I'm thinking a user could accept an invitation and install an app multiple times.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add an extra sentence that defines "acceptance" so it's clear it's per-installation and not per-user?

An acceptance is when a user installs an app from your invitation; multiple installations count as multiple acceptances.
Invitations are limited to a maximum of 500 acceptances by default and are valid for 30 days.
Once either the accept limit or the time limit is reached, you will need to create a new invitation.
Leave blank to accept the default limit of 500, or enter a different acceptance limit if desired.

@rossiam rossiam force-pushed the schema-limit-update branch from 99d540f to 15f1147 Compare January 11, 2024 21:15
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