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 subject overhaul #359

Closed
wants to merge 3 commits into from
Closed

Shorten subject overhaul #359

wants to merge 3 commits into from

Conversation

eurich
Copy link
Member

@eurich eurich commented May 1, 2013

small improvements to things that annoyed me for a long, long time ;P

  • shorten_subject renamed to shorten_string
  • moved hardcoded parameters to elkarte_settings

Note: There's currently no GUI implemented, I doubt it will be changed that often but if someone wants to change it: It's now possible without file edits.

eurich added 3 commits May 1, 2013 18:20
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
…ettings

Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
… via hidden datatase flags.

Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
@norv
Copy link
Contributor

norv commented May 4, 2013

The PR has got conflicts with master in the meantime, TE. Sorry about that. Could you please look into it, merge or rebase eventually?

@eurich
Copy link
Member Author

eurich commented May 4, 2013

no problem at all, will fix the conflicts later and rebase ..

@eurich
Copy link
Member Author

eurich commented May 4, 2013

uuups, too many conflicts.. don't know what happend.. closing the PR for now, I'll rebuild it after #366 gets merged.

@eurich eurich closed this May 4, 2013
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