Feedback

[julien-nc]

• One eslint warning left about src/components/PersonalSettings.vue line 20 (displayed when compiling or with npm run lint)
• IMO The URL should be stored as set by the user (no stripping + adding trailing slash). So the value loaded after a page refresh is not different than what the user has set. The trailing slash should only be added when the URL is used to make a network request (calling the API and redirecting the user). Wdyt?
• Could you check if it's possible to display something informative in the dashboard widget if the app is not configured?
• IMO the API token should be "validated" when setting a new value. One simple and useful way to check it would be to use it to get information about the documenso account. Then you can even store this information (display name, user ID etc...) and display it in the settings. You can even hide the API token input and replace it with a "disconnect" button which clears the token value and deletes all the user info.
• You could pass the info to the frontend (initial state or request when opening the modal) that the user is connected or not (valid api token) so that the modal could display something if no api token is set (or it's not valid). You could just say: "Nope, not connected to any documenso account, check the connected account settings" + a link/button to get there.