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

Recommendation for Warnings after installation/update #327

Closed
q2apro opened this issue Nov 7, 2017 · 7 comments
Closed

Recommendation for Warnings after installation/update #327

q2apro opened this issue Nov 7, 2017 · 7 comments

Comments

@q2apro
Copy link

q2apro commented Nov 7, 2017

This was shown after the update:

02

I could quickly find the "File Manager Path" in the Admin settings. However, I could not find the "File Base Path"... just after a while I realized that it is hidden under "File Browser" (which is always "No" in my installation, so settings are hidden).

I would really like to see those two warnings on top linked to the option, the underlined words.

Thanks!

@sashabeep
Copy link

sashabeep commented Nov 7, 2017

That's amazing! wait for this message for 10 years, so glad to see it in any form 👍 So links is not the main goal, reset paths is the first step after moving site to another place http://www.evolution-docs.com/documentation/administration/moving-site

@q2apro
Copy link
Author

q2apro commented Nov 8, 2017

FYI, I did not move the site to another place. I just ran the update.

@fourroses666
Copy link

fourroses666 commented Nov 8, 2017

Or add the text;

Please check and update your Evo system configuration. You can reset the file base path under Configuration > File Browser.

and for manager .... File Manager

A link to configuration /manager/#?a=17 but I don't think its possible to open the tab

By the way, this would help too, reset the paths after upgrading:
#159

@Nicola1971
Copy link

@fourroses666
/manager/#?a=17&tab=5 for File Manager
/manager/#?a=17&tab=6 for File Browser

should work, if you don't have additional tabs (ie: plugin tabs in settings)

@fourroses666
Copy link

fourroses666 commented Nov 8, 2017

All lang files need to be changed like this;

<u>File Manager path</u>
<u>File base path</u> needs to be

<a href="/manager/#?a=17&tab=5">File Manager path</a>
<a href="/manager/#?a=17&tab=6">File base path</a>

https://github.com/evolution-cms/evolution/blob/develop/manager/includes/lang/english.inc.php#L129
https://github.com/evolution-cms/evolution/blob/develop/manager/includes/lang/english.inc.php#L142

Maybe someone can create the pull?

@Nicola1971
Copy link

@fourroses666, A link to system settings is not a solution, if you have hidden file browser tab like @q2apro
Better solution: add a "reset path" button to the alert, like the reset in the alert of modified system files

@Dmi3yy
Copy link

Dmi3yy commented Nov 8, 2017

Done but, not manager ) [+MGR_DIR+]

Its in 99% show after Update or Move site then first login is Admin )

@Dmi3yy Dmi3yy closed this as completed Nov 8, 2017
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

No branches or pull requests

5 participants