Skip to content
This repository has been archived by the owner on Mar 27, 2022. It is now read-only.

links components collection #116

Open
atherdon opened this issue Jan 22, 2021 · 16 comments
Open

links components collection #116

atherdon opened this issue Jan 22, 2021 · 16 comments
Assignees

Comments

@atherdon
Copy link
Owner

atherdon commented Jan 22, 2021

мне кажется я гдже то это или упоминал или описывал.

смысл в чем. у нас будет много ссылок, и чаще всего они будут захардкожены, но могут и менятся.
поэтому я бы хотел, чтобы у нас, на основе появились свои мини компоненты.
у нас уже есть:

  • ссылки в header
  • ссылки More в cards
  • будут просто ссылки на внутренние папки и т.д.
  • есть ссылки на наши articles
  • будет ссылка на https://plagiarismdetector.net/
  • будет ссылка mailto

и тд и тд

импрув может быть и небольшой, но в дальнейшем он нам сохранит немного нервов.

но это очень простой таск, я думаю Меруж должен его затянуть.

@atherdon
Copy link
Owner Author

#113

@atherdon
Copy link
Owner Author

@atherdon
Copy link
Owner Author

@coder-do similar task, related to links

@coder-do
Copy link
Collaborator

@atherdon вы такое имеете ввиду ? Создать массивы из ссылок и роутов потом их импортировать туда , где понадобится

chrome_fliUTqFGtN

@atherdon
Copy link
Owner Author

atherdon commented Jan 30, 2021 via email

@atherdon
Copy link
Owner Author

atherdon commented Jan 30, 2021 via email

@coder-do
Copy link
Collaborator

@atherdon не очень понял, что нам щас нужно,я так понял массив сделать с роутами и ссылками отдельно, или таск не очень понял.
Вы какие ссылки хотите отдельно вынести? И их просто вынести в функции которые возвращают эти ссылки ( для каждой ссылки допустим 1 компонент и её вызывать где понадобится) ?

@atherdon
Copy link
Owner Author

ок.
мою проблему очень просто проилюстрировать.
я незнаю сколько страниц и ссылок у нас есть на этомпроекте.
уверен что остальные девы тоже затруднятся ответить быстро.
Вы можете сами попробывать пооткрывать код и заметите что работа со ссылками размазана по всей тарелке.
чтобы както контролировать эту ситуацию - нам нужно одно основное место - хранилище, которое можно будет открыть и в этом одном файле мы увидим всю картину.

задание возможно неточное. но и его нужгно делать

@atherdon
Copy link
Owner Author

cc @coder-do

@coder-do coder-do self-assigned this Feb 1, 2021
@coder-do
Copy link
Collaborator

coder-do commented Feb 2, 2021

Ссылки типа https://app.hackernoon.com/drafts/4ub652BZlv0353orv7XU и mailto:xxx.com они находятся в папке data/brands.js и data/top-authors.js - их выносить думаю не стоит пусть там и останутся

@atherdon
Copy link
Owner Author

atherdon commented Feb 2, 2021

@coder-do я бы их тоже вынес

@atherdon
Copy link
Owner Author

atherdon commented Feb 2, 2021

например мы решим добавить трекинг того, как часто редакторы кликают эти ссылки

@coder-do
Copy link
Collaborator

coder-do commented Feb 2, 2021

они в вот таком виде хранятся в дата файлах , если эти ссылки вынести потом нужно в эти файлы импортировать и в draft url, email вставить во всех местах, чтоб остальной функционал не ломался, т.к они сразу идут в пропсы для таблиц
{ 'Draft url': 'https://app.hackernoon.com/drafts/4oJ3dI9QGwqwHuhBYT8F', '': '', v: 'v', email: 'mailto:xxx.com', Published: 'TRUE', Added: 4.09, Edited: 7.09, Published__1: 7.09, 'Avg time story': 4, 'Avg time ALL story': 6, __1: '' },

@atherdon
Copy link
Owner Author

atherdon commented Feb 2, 2021

тогда нам нужно придумать как мы сможем редактировать json который мы отправляем в таблицу.
потому что я планирую добавить больше ссылок

сс @coder-do

@coder-do
Copy link
Collaborator

coder-do commented Feb 3, 2021

Окей подумаю над задачей.

@vadim9999
Copy link
Collaborator

@coder-do Hi! How are you?

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

No branches or pull requests

3 participants