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

Больше деталей #3

Merged

Conversation

TatianaSenatorova
Copy link
Contributor

@TatianaSenatorova TatianaSenatorova commented Oct 28, 2024

@TatianaSenatorova
Copy link
Contributor Author

Артем, привет. Извини, пришлось вставить console.log, передать туда полученный массив и исключить этот console.log из линтера. Так как у нас полученный в задании массив нигде не используется, то не проходило проверкой линтерами. Только так получилось отправить на проверку.

@keksobot keksobot changed the title Module4 task1 Больше деталей Oct 28, 2024
Copy link

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Чуть чуть давай переделаем, но в остальном отличная работа!


const createPhoto = (index = photoCounter()) => ({
id: `${index}`,
url: `photos/${index}.jpg`,
Copy link

Choose a reason for hiding this comment

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

у нас есть всего 6 фотографий, давай здесь вставим рандомное число от 1 до 6, а не index

js/main.js Outdated
const createPhoto = (index = photoCounter()) => ({
id: `${index}`,
url: `photos/${index}.jpg`,
description: DESCRIPTIONS[index],
Copy link

Choose a reason for hiding this comment

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

здесь тоже давай вставим рандомную фразу из этого списка. Index будем использовать для уникального id

@keksobot keksobot merged commit 08922e3 into htmlacademy-javascript:master Oct 30, 2024
1 check passed
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.

3 participants