-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: revamp profile pages #145
Conversation
e1da960
to
4175875
Compare
e10917a
to
534be62
Compare
The features from this PR is done, but unfortunately the |
646ff7c
to
d35c649
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
<NuxtLink :class="css.link" to="/checkout/plan"> | ||
premium subscription plan. | ||
</NuxtLink> | ||
<div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happens if we skip the extra div here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
components/account/home/ApiKeys.vue
Outdated
:class="css.input" | ||
<div class="grid md:grid-cols-2 gap-4"> | ||
<RuiRevealableTextField | ||
id="username" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have to change the id here and below :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Another thing is that there is no visual feedback at all on regenerate of API keys. The first time it is obvious that something happened but after that there is no indication. |
Another thing that seems to be broken is the redirect link to home after buying a subscription. |
633c11b
to
fd9c8dd
Compare
fd9c8dd
to
8580c2d
Compare
Co-authored-by: Joseph Ojoko <entewshi@gmail.com>
8580c2d
to
f8ad2d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me
No description provided.