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

Reorganize tags #1121

Merged
merged 4 commits into from
Mar 9, 2023
Merged

Conversation

IstoraMandiri
Copy link
Collaborator

@IstoraMandiri IstoraMandiri commented Mar 3, 2023

As discussed with @DonaldMcIntyre , this PR attempts to consolidate and standardize the tags we use on blog articles.

This PR is provisional and open to suggestions, I have made some opinionated decisions but am happy to edit if needed.

Some notes:

  • I ensured all tags are lowercase singular (e.g. "Guides" -> "guide").
  • "news", "videos" and "apps" are auto-generated, they are reserved and shouldn't be used, so these tags have been removed from any blog posts using them
  • "course" is removed, instead I have added the tag "series", which can be used for all articles that are part of a series. They can also be tagged with "guide".
  • Added series tag to the "How to contribute to ETC" series
  • Rename instances of "tutorial" to legacy "guide"
  • Remove "infrastructure" tag
  • Remove "opinion" tag, but add opinion disclaimer for those tagged "opinion", added "philosophy" tags to those only tagged as opinion
  • For posts that had changes, ensure Chinese tags and disclaimers are the same as English

From now on, please use one or more of the listed tags as below (using the lowercase version). If a new tag is needed, please open a PR to add that tag first.

  announcement: Announcements
  application: Apps
  development: Development
  education: Education
  event: Events
  exchange: Exchanges
  guide: Guides
  hardfork: Hard Forks
  media: Media
  mining: Mining
  podcast: Podcasts
  teams: Teams
  trading: Trading
  wallet: Wallets
  philosophy: Philosophy
  series: Series

TODOs:

@netlify
Copy link

netlify bot commented Mar 3, 2023

Deploy Preview for ethereumclassic ready!

Name Link
🔨 Latest commit 4392e89
🔍 Latest deploy log https://app.netlify.com/sites/ethereumclassic/deploys/6408c993c348a400073a0bc9
😎 Deploy Preview https://deploy-preview-1121--ethereumclassic.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@DonaldMcIntyre
Copy link
Contributor

"course" is removed, instead I have added the tag "series", which can be used for all articles that are part of a series. They can also be tagged with "guides".

guides -> guide

For posts that had changes, ensure Chinese tags and disclaimers are the same as English

I don't know how @angelahylliu will identify all the ones that had tag changes, but I ping her so she can work on it.

@IstoraMandiri
Copy link
Collaborator Author

IstoraMandiri commented Mar 4, 2023

guides -> guide

Thanks, fixed.

I don't know how @angelahylliu will identify all the ones that had tag changes, but I ping her so she can work on it.

Sorry for miscommunication, I meant that I applied the changes to the existing Chinese posts. Going forward, just have the Chinese version copy the English tags (as you were doing).

@gitr0n1n
Copy link
Contributor

gitr0n1n commented Mar 8, 2023

Good to go when you're ready @IstoraMandiri

@IstoraMandiri IstoraMandiri merged commit cfadcec into ethereumclassic:master Mar 9, 2023
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.

4 participants