LBRYCommunity/lbry-redux
1
0
Fork 0
Code Issues 5 Pull requests 6 Projects Releases Packages Wiki Activity

Incorporating default tags in spanish #321

Merged
michaelmitnick merged 3 commits from master into master 2020-06-05 17:12:08 +02:00
Conversation 1 Commits 3 Files changed 2 +28 -16
2 changed files with 28 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
dist/bundle.es.js vendored
View file

File diff suppressed because one or more lines are too long

24
src/constants/tags.js
View file

@ -36,7 +36,7 @@ export const MATURE_TAGS = [
'hentai',
];
export const DEFAULT_KNOWN_TAGS = [
const DEFAULT_ENGLISH_KNOWN_TAGS = [
'free speech',
'censorship',
'gaming',
@ -521,3 +521,25 @@ export const DEFAULT_KNOWN_TAGS = [
'covidcuts',
'covid-19',
];
const DEFAULT_SPANISH_KNOWN_TAGS = [
'español',
'tecnología',
'criptomonedas',
'economía',
'bitcoin',
'educación',
'videojuegos',
'música',
'noticias',
'ciencia',
'deportes',
'latinoamérica',
'latam',
kauffj commented 2020-06-04 00:40:50 +02:00 (Migrated from github.com)
Review
Copy link

@seanyesmunt I told @michaelmitnick to do this but we should probably split the Spanish tags into a separate array for easier future refactoring/cleanup

@seanyesmunt I told @michaelmitnick to do this but we should probably split the Spanish tags into a separate array for easier future refactoring/cleanup
kauffj commented 2020-06-04 00:41:12 +02:00 (Migrated from github.com)
Review
Copy link

(and call .unique() on resultant merge)

(and call `.unique()` on resultant merge)
'conspiración',
'humor',
'política',
'tutoriales',
];
export const DEFAULT_KNOWN_TAGS = [...DEFAULT_ENGLISH_KNOWN_TAGS, ...DEFAULT_SPANISH_KNOWN_TAGS];