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

Update export.queries.php #1840

Closed
wants to merge 2 commits into from
Closed

Update export.queries.php #1840

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 27, 2017

If I have TeamPass installed on a machine with an IP address equal to 255.255.255.255, the link referenced in the "href" attribute (generated when using the offline mode) points to "http://255.255.255.255/teampass/files/...".
For users with less technological knowledge, what will probably happen is that they'll open the link and write down the generated URL on a paper. In the future, when trying to use it without an internet connection, it won't work.

My suggestion is to add the "download" attribute to the HTML "a" tag created. This attribute allows to download the file referenced in the "href" attribute when clicking on the link. This way, it'll be more clear that a file was created and users need to store it on a USB pen, for example, in order to be able to use it in the future.

Added the "download" attribute. This attribute allows to download the file referenced in the "href" attribute when clicking on the link.

In my opinion, as it was, the process of obtaining a file that could be used offline was not transparent... For less tech savvy users, it could lead to some problems.
@nilsteampassnet
Copy link
Owner

Thank you for the tip.

nilsteampassnet added a commit that referenced this pull request Nov 11, 2017
Merged PR #1840
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.

None yet

2 participants