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

[Self-hosted]: cannot search users/workspaces without enterprise license and error is console only #6932

Closed
Tracked by #7597
mrsimonemms opened this issue Nov 26, 2021 · 6 comments · Fixed by #8593
Closed
Tracked by #7597
Assignees
Labels
feature: admin dashboard needs visual design team: webapp Issue belongs to the WebApp team type: bug Something isn't working

Comments

@mrsimonemms
Copy link
Contributor

Bug description

When I go to /admin/users in my self-hosted instance and click "search", I'm not able to search without an enterprise license. Whilst this is fine, the error only appears in the console.

It would be nice to have the error in the actual dashboard - an alert-style error

image

Steps to reproduce

Install self-hosted, go to admin and click "search"

Workspace affected

No response

Expected behavior

Should display the "enterprise license required" in the dashboard, not just the console

Example repository

No response

Anything else?

No response

@csweichel csweichel added feature: admin dashboard team: webapp Issue belongs to the WebApp team labels Nov 30, 2021
@JanKoehnlein
Copy link
Contributor

@jldec Shouldn't we rather disable the admin routes for non enterprise installs?

@jldec
Copy link
Contributor

jldec commented Dec 1, 2021

Either way works - a visible nudge can help to make users aware that they need a license :)

Silent browser console errors should be treated more like bugs. It would be helpful from an operations perspective if we had a way to collect those, just like server-side logs.

@laushinka
Copy link
Contributor

Initially I would go for removing the Admin tab from non-enterprise users, but a visible nudge also makes sense.
How about showing a disabled Admin tab with a locked icon, and a tooltip saying that it's only for Professional users? Probably there are better ideas, but something along this line?

@jldec jldec moved this to Needs Design in 🍎 WebApp Team Jan 17, 2022
@gtsiolis gtsiolis moved this to This Week in Gitpod Design Jan 20, 2022
@gtsiolis gtsiolis moved this from This Week to Inbox in Gitpod Design Jan 20, 2022
@gtsiolis gtsiolis removed this from Gitpod Design Feb 3, 2022
@jankeromnes
Copy link
Contributor

Suggestion: The visible nudge to indicate that a license is required could be done like so:

  1. Catch enterprise license required errors in Admin pages

  2. Add export in front of the LimitReachedModal:

function LimitReachedModal(p: { children: React.ReactNode }) {
const { user } = useContext(UserContext);
return <Modal visible={true} closeable={false} onClose={() => { }}>
<h3 className="flex">
<span className="flex-grow">Limit Reached</span>
<img className="rounded-full w-8 h-8" src={user?.avatarUrl || ''} alt={user?.name || 'Anonymous'} />
</h3>
<div className="border-t border-b border-gray-200 dark:border-gray-800 mt-4 -mx-6 px-6 py-2">
{p.children}
</div>
<div className="flex justify-end mt-6">
<a href={gitpodHostUrl.asDashboard().toString()}><button className="secondary">Go to Dashboard</button></a>
<a href={gitpodHostUrl.with({ pathname: 'plans' }).toString()} className="ml-2"><button>Upgrade</button></a>
</div>
</Modal>;
}

  1. Import, customize (text and call-to-action), and show this modal -- will look something like this:

Source

@jldec
Copy link
Contributor

jldec commented Mar 3, 2022

Note discussion in #8328

could render this issue moot:

I would suggest that the admin dashboard is enabled for all license levels. This will allow us to put useful things in here in future

Even if we do keep limiting the use of the admin dashboard, the check should allow evaluation use when there are < 10 users.

@mrsimonemms
Copy link
Contributor Author

As part of #8593, I've made the admin dashboard available to all licence levels. Once accepted, I think that solves this ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: admin dashboard needs visual design team: webapp Issue belongs to the WebApp team type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants