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

Добавляет название календаря #492

Merged
merged 2 commits into from
Jan 31, 2020
Merged

Добавляет название календаря #492

merged 2 commits into from
Jan 31, 2020

Conversation

mbelsky
Copy link
Contributor

@mbelsky mbelsky commented Jan 31, 2020

Привет,

Названия и описания не было в задаче, взял из README.

Закрывает #428

@pepelsbey
Copy link
Member

Спасибо! А как это в итоге выглядит в итоговом календаре? Если запустить сборку.

@@ -36,7 +36,9 @@ function prepareEvent(event) {

function generateICalCalendar(events) {
const cal = ical({
description: 'Конференции, встречи и другие события по фронтенду во всём мире в одном календаре.',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
description: 'Конференции, встречи и другие события по фронтенду во всём мире в одном календаре.',
description: 'Конференции, встречи и другие события по фронтенду во всём мире.',

domain: 'https://web-standards.ru/',
name: 'Календарь событий по фронтенду',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name: 'Календарь событий по фронтенду',
name: 'Фронтенд',

Copy link
Member

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Хорошо, когда у календарей короткие названия, чтобы в интерфейсе они не торчали и не мешали.

imageimage

@mbelsky
Copy link
Contributor Author

mbelsky commented Jan 31, 2020

Спасибо! А как это в итоге выглядит в итоговом календаре? Если запустить сборку.

@pepelsbey первые строчки .ics файла:

Текущая версия

BEGIN:VCALENDAR	
VERSION:2.0	
PRODID:-//web-standards//calendar//RU	
BEGIN:VEVENT

С изменениями (без внесения текстовых правок)

	BEGIN:VCALENDAR
VERSION:2.0
PRODID:-//web-standards//calendar//RU
NAME:Календарь событий по фронтенду
X-WR-CALNAME:Календарь событий по фронтенду
X-WR-CALDESC:Конференции, встречи и другие события по фронтенду во всём ми
 ре в одном календаре.
BEGIN:VEVENT

Полный diff: https://www.diffchecker.com/yv1Ym4zu

С внесением текстовых правок

BEGIN:VCALENDAR
VERSION:2.0
PRODID:-//web-standards//calendar//RU
NAME:Фронтенд
X-WR-CALNAME:Фронтенд
X-WR-CALDESC:Конференции, встречи и другие события по фронтенду во всём ми
 ре.
BEGIN:VEVENT

Конец строки X-WR-CALDESC: переносится на следующую строку , не знаю на сколько это критично

@pepelsbey
Copy link
Member

Укоротил название, надеюсь теперь не будет переноситься, на всякий случай :)

@pepelsbey pepelsbey merged commit 164f3be into web-standards-ru:master Jan 31, 2020
@pepelsbey
Copy link
Member

Спасибо!

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.

2 participants