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

Wiki tagging #407

Merged
merged 12 commits into from
Dec 31, 2022
Merged

Wiki tagging #407

merged 12 commits into from
Dec 31, 2022

Conversation

frankystone
Copy link
Contributor

@frankystone frankystone commented Dec 20, 2022

Fixes #323

The tagging app was already used by the wiki. What was missing were some views and modified html to make the tagging really useful.

  • Added an input field Tags to a wiki edit page
  • Show tags for each article on the index page
  • Linkify tags and added a view to show all articles where this tag is used, also with additional tags related to the shown article(s)
  • articles with tag 'outdated' gets an additional text on top

Sometimes the tags are displayed with an uppercase letter. Don't know why yet.

The used tagging app looks not maintained anymore and it looks like it is not ready for Django version > 4. Probably it is better to use django-taggit?

Wiki Index with tags to each article, of course one article can have more than one tag:
tagging_01

All articles with tag 'atlanteans', one page has an additional tag 'Story':
tagging_03

Hint about outdated wiki page:
tagging_04

Copy link
Member

@Noordfrees Noordfrees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 😄
Some phrasing suggestions inline

wiki/templates/wiki/tag_view.html Outdated Show resolved Hide resolved
wiki/templates/wiki/view.html Outdated Show resolved Hide resolved
wiki/templates/wiki/index.html Show resolved Hide resolved
@blotter
Copy link
Contributor

blotter commented Dec 20, 2022

läuft auf alpha zum testen

@frankystone
Copy link
Contributor Author

A description on how to use tagging can be found here: https://alpha.widelands.org/wiki/WikiHelp/#tagging

@frankystone
Copy link
Contributor Author

@Noordfrees are you fine with my string changes?

@Noordfrees
Copy link
Member

Texts LGTM :)

@blotter blotter merged commit a6a8183 into master Dec 31, 2022
@blotter blotter deleted the wiki_tagging branch December 31, 2022 22:56
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.

Outdated Wiki Pages
3 participants