Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Changed workflows categories to lowercase and removed dashes #244

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

andraz213
Copy link
Contributor

@andraz213 andraz213 commented Mar 25, 2021

Fixes #185 and fixes potential future problems with dashes on the blog site.

Fixes #207

@andraz213
Copy link
Contributor Author

Fixed #207 as well while I'm at it.

@andraz213
Copy link
Contributor Author

Blog categories now show the active category. It was only implemented on the workflows site and not on the blog site, so I decided to add it.

@markotoplak
Copy link
Member

Thanks. How did you handle old URLs? Are there any redirects if someone perhaps had an old URL bookmarked? Did you search the current site if the old URLs were used anywhere?

@andraz213
Copy link
Contributor Author

andraz213 commented Mar 26, 2021

Good point. I can change the URLs back and just change how categories are displayed.
Redirects could be implemented, but I don't know how this site is hosted.

@andraz213
Copy link
Contributor Author

The URLs are now the same as before. Just the tags are displayed in lowercase letters.

@irgolic
Copy link
Member

irgolic commented Mar 30, 2021

LGTM, please rebase and solve conflicts following merge of #245

@markotoplak Can you merge this, if you're okay with the aforementioned URL redirects?

@markotoplak
Copy link
Member

@andraz213, sorry, I somehow overlooked your comment. Should be fine now, could you please rebase?

@ajdapretnar
Copy link
Collaborator

@markotoplak @irgolic Could we add the auto-rebase option which we have in orange3 (if I am not mistaken)?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflows: current tag is not selected if it is multiple-word Put workflow tags in lowercase and remove dash
4 participants