mbin icon indicating copy to clipboard operation
mbin copied to clipboard

Rename remaining article to thread

Open melroy89 opened this issue 1 year ago • 2 comments

Rename all the remaining keywords from article(s) to thread(s). Making the code more consistent.

Which also means I renamed the new valid link for creating a new thread: new/thread in the URL (same for the API).

This PR also finally solves the translation strings that then where still have the key article, articles, edit_article (which was always missing btw), add_new_article, etc. To the new keys: type.thread, thread, threads, add_new_thread and finally edit_thread.

To be backwards compatible with existing data in the database (specifically the type column in the entry table), we keep the article string in the resolveType() function, so t he old string "article" or "articles" are still valid. Same for some Twig templates, we extend with another or for backwards compatibly for now.

The next logical follow-up would be to rename entry as well later.

melroy89 avatar Sep 19 '24 18:09 melroy89

Ok I will look at it closer tomorrow. Code wise it looks good now

BentiGorlich avatar Sep 20 '24 15:09 BentiGorlich

Ok I will look at it closer tomorrow. Code wise it looks good now

This pr is less urgent than the 1.7.x fixes. No worries here. It's not really a bug fix.

melroy89 avatar Sep 20 '24 15:09 melroy89

Rebased

melroy89 avatar Nov 08 '24 13:11 melroy89

@melroy89 do you have an idea why there are so many commits linked to this PR? Doesn't really make sense to me especially since there are multiple commits of other PRs

BentiGorlich avatar Nov 24 '24 14:11 BentiGorlich

@melroy89 do you have an idea why there are so many commits linked to this PR? Doesn't really make sense to me especially since there are multiple commits of other PRs

yeah, I also found this weird. Maybe something went wrong with merging/rebasing?

melroy89 avatar Nov 24 '24 17:11 melroy89

I might need to redo this all, and revert several commits via reset hard or something. And try to rebase again. Maybe I started on the main branch initially? And later changed to the new dev/new_features branch?

melroy89 avatar Nov 24 '24 17:11 melroy89