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

[3pt] - Add all items counter on the Homepage - Student A #6

Open
Benawi opened this issue May 29, 2023 · 0 comments · Fixed by #34
Open

[3pt] - Add all items counter on the Homepage - Student A #6

Benawi opened this issue May 29, 2023 · 0 comments · Fixed by #34

Comments

@Benawi
Copy link
Owner

Benawi commented May 29, 2023

Even if the API gives you the number of items, you will create a specific function to calculate it.

Make sure that the correct number is displayed on the Homepage.

Respect the following rules:

  • Each counter should be implemented as a separate module.
  • A counter function should look for specific DOM elements (e.g. for the comments counter it should look for comments) and make the counting based on what is actually displayed on the page.
  • A counter function should cover all the edge cases you can think about.
@Megagig Megagig self-assigned this May 29, 2023
@Benawi Benawi moved this from To do to In progress in [COPY] [TEMPLATE] JavaScript Group Capstone May 30, 2023
@Benawi Benawi moved this from In progress to To do in [COPY] [TEMPLATE] JavaScript Group Capstone May 30, 2023
@Benawi Benawi moved this from To do to In progress in [COPY] [TEMPLATE] JavaScript Group Capstone May 31, 2023
@Benawi Benawi moved this from In progress to To do in [COPY] [TEMPLATE] JavaScript Group Capstone May 31, 2023
@Benawi Benawi moved this from To do to In progress in [COPY] [TEMPLATE] JavaScript Group Capstone May 31, 2023
@Megagig Megagig moved this from In progress to In review in [COPY] [TEMPLATE] JavaScript Group Capstone May 31, 2023
@Benawi Benawi moved this from In review to Done in [COPY] [TEMPLATE] JavaScript Group Capstone May 31, 2023
@Benawi Benawi linked a pull request Jul 6, 2023 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants