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

Updated urls to OSGeoLive docs (#1500) #22

Merged
merged 5 commits into from
Jul 14, 2015
Merged

Conversation

kalxas
Copy link
Member

@kalxas kalxas commented Jul 9, 2015

No description provided.

@johanvdw
Copy link
Member

johanvdw commented Jul 9, 2015

Looking at it this way, shouldn't we remove html from the path,
so instead of:
http://localhost/osgeolive/html/en/quickstart/52nWPS_quickstart.html
http://localhost/osgeolive/en/quickstart/52nWPS_quickstart.html

I can just adapt the symlink.

johanvdw added a commit that referenced this pull request Jul 9, 2015
As proposed in a comment at: #22 -
Use eg
http://localhost/osgeolive/en/quickstart/52nWPS_quickstart.html
instead of:
http://localhost/osgeolive/html/en/quickstart/52nWPS_quickstart.html
johanvdw and others added 2 commits July 9, 2015 19:46
As proposed in a comment at: OSGeo#22 -
Use eg
http://localhost/osgeolive/en/quickstart/52nWPS_quickstart.html
instead of:
http://localhost/osgeolive/html/en/quickstart/52nWPS_quickstart.html
@kalxas
Copy link
Member Author

kalxas commented Jul 9, 2015

@johanvdw please review

@johanvdw
Copy link
Member

johanvdw commented Jul 9, 2015

I will merge, but I'm just wondering if we should have an index.html also at http://localhost pointing to http://localhost/osgeolive

@kalxas
Copy link
Member Author

kalxas commented Jul 9, 2015

The desktop link is fixed to point to http://localhost/osgeolive
It would be nice to find a way to have that link as default in firefox.

@johanvdw
Copy link
Member

johanvdw commented Jul 9, 2015

I'll research that.

@kalxas
Copy link
Member Author

kalxas commented Jul 10, 2015

Just tested latest nightly build and the link issue works fine.
The image scaling issue remains, but we should open another ticket for that.
@johanvdw can you please merge?

johanvdw added a commit that referenced this pull request Jul 14, 2015
Updated urls to OSGeoLive docs (#1500)
@johanvdw johanvdw merged commit 3398b6e into OSGeo:master Jul 14, 2015
@kalxas kalxas deleted the doc_fix branch July 14, 2015 15:26
This pull request was closed.
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.

2 participants