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

Provide quota limit feedback to users #1594

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

dustinbyrne
Copy link
Contributor

@dustinbyrne dustinbyrne commented Feb 14, 2024

This is a work in progress.

TODO:

  • Check the current quota before a chat has begun
  • Add automated tests

image

It's possible that the backend does not permit any free daily messages. If so:
image

Copy link

AppMap runtime code review

Summary Status
Failed tests ✅ All tests passed
API changes 0️⃣ No API changes
Security flaws ✅ None detected
Performance problems ✅ None detected
Code anti-patterns ✅ None detected
New AppMaps 0️⃣ No new AppMaps

<div class="options">
<div class="option" v-if="limit > 0">
<div class="option__grant center">Continue on the free plan</div>
<p>The free plan allows you to send up to {{ limit }} messages to Navie AI per day.</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe “make up to X requests” or “have up to X conversations”

<p>AppMap Pro has no daily restriction on the number of daily messages sent to Navie AI.</p>
<p>Upgrading will instantly restore access.</p>
<a
href="https://billing.stripe.com/p/login/14kcOoc0gbrIdqMeUU"
Copy link
Contributor

Choose a reason for hiding this comment

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

We should have a page on getappmap.com for this because they may want to choose a personal subscription or a team subscription - or even Enterprise. So maybe “Compare plans”

class="center"
target="_blank"
>
<v-button label="Subscribe" size="medium" class="subscribe" />
Copy link
Contributor

Choose a reason for hiding this comment

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

“Compare plans”?

@kgilpin
Copy link
Contributor

kgilpin commented Oct 9, 2024

@dustinbyrne this is a useful design, and I think we have the data we need for this now in the ConversationThread response.

We would need to handle the case where the thread enrollment response is an error, due to e.g. proxy settings or the Navie services being down. We would not want that condition to block the use of Navie.

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