Skip to content

docs: Update outdated plugin documentation#770

Merged
yamgent merged 1 commit into
MarkBind:masterfrom
jamos-tay:plugins-doc-fix
Mar 24, 2019
Merged

docs: Update outdated plugin documentation#770
yamgent merged 1 commit into
MarkBind:masterfrom
jamos-tay:plugins-doc-fix

Conversation

@jamos-tay

Copy link
Copy Markdown
Contributor

What is the purpose of this pull request? (put "X" next to an item, remove the rest)

• [X] Documentation update

What is the rationale for this request?

Somehow during the documentation restructuring the filterTags documentation was changed to the older, incorrect one (#474). It should be the one from #689.

What changes did you make? (Give an overview)

Replaced it with the newer one (#689)

@yamgent yamgent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • siteConfiguration.md still shows the wrong way of using "tags" settings.
  • This part of the code may not be very clear that it is inside "filterTags"?

```json
site.json
{
...
"tags": ["-language--*", "language--C#"]
}
```
`language--java` is overridden by `-language--*`, so only `language--C#` is shown.

@jamos-tay

Copy link
Copy Markdown
Contributor Author

Updated

@yamgent yamgent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Apologies for the late review

Comment thread docs/userGuide/siteConfiguration.md Outdated
@jamos-tay

Copy link
Copy Markdown
Contributor Author

Updated, rebased

@yamgent yamgent added this to the v1.21.1 milestone Mar 23, 2019
@yamgent yamgent merged commit a58a410 into MarkBind:master Mar 24, 2019
@yamgent yamgent changed the title Fix tagging documentation under plugins docs: Update outdated plugin documentation Mar 24, 2019
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