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

Shorten external pages urls (Trac #3235) #3235

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 7 comments
Closed

Shorten external pages urls (Trac #3235) #3235

elgg-gitbot opened this issue Feb 16, 2013 · 7 comments

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/3235 on 41228859-04-11 by ewinslow, assigned to unknown.

Elgg version: 1.7

Currently the externalpages plugin registers pages at expages/read/About (or Terms, or Privacy, etc...). I would love to see it rewritten to just /terms, /about, /privacy.

@elgg-gitbot
Copy link
Author

Attachment added by trac user pouyanster on 41239201-02-24: 3235_shorten_external_pages_urls.diff

@elgg-gitbot
Copy link
Author

trac user pouyanster wrote on 41239204-07-16

As of yesterday, no one was working on this issue, so I started it yesterday and finished it today! I hope that it's ok.

@elgg-gitbot
Copy link
Author

cash wrote on 41259630-10-25

(In [svn:8943]) Fixes #3235 integrated Kishanthan's commits for cleaner site pages urls

@elgg-gitbot
Copy link
Author

cash wrote on 41259636-07-08

Sorry, pouyanster, Kishanthan had been working on this one. Thanks for the effort. It looks like your commit was done on 1.7 but we using svn trunk for active development.

@elgg-gitbot
Copy link
Author

trac user pouyanster wrote on 41260414-08-21

Replying to cash:

Sorry, pouyanster, Kishanthan had been working on this one. Thanks for the effort. It looks like your commit was done on 1.7 but we using svn trunk for active development.

I tried to fix the bug as a part of "Got Skill?" for GSoC. It would have been nice to inform me sonner so I could start working on another project!
Regards

@elgg-gitbot
Copy link
Author

cash wrote on 41261270-12-17

Replying to pouyanster:

Sorry for the delay in the response, pouyanster. We're still about to learn something about your coding skills from the patch so that is good.

@elgg-gitbot
Copy link
Author

Milestone changed to Elgg 1.8 Beta by cash on 42939899-06-04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant