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

Feature/creatives and sectors #12

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

laz95
Copy link

@laz95 laz95 commented Sep 28, 2023

  • Added Creatives table to Library page:
    • Added a new creative page that uses creatives domain to retrieve creatives based on the current brand id retrieved from session state
    • Added a new creatives table page that contains html&css for the table and uses TableUI component
    • Added Creatives repository that calls listFolder in schema that retrieves creatives from backend
    • Added Creatives domain that calls creatives repository
  • Added date formatter tool and unit test to format date like dd/mm/yyyy
  • Added Sectors page
    • Added new Sectors page that for every sector retrieved creates a Sector widget and renders it on the page
    • Created Sectors widget with html&css for sectors responsive block
    • Added sectors repository that retrieves sectors names and counts from mocked backend response in schema-mocs.ts
    • Added sectors domain that creates an array of Sector objects that contain sectorId, name and count
  • Added a spinner on Sectors page load that simulates retrieving data for 1000ms based on isLoading information in session state. Resused LoadingPage that already exists.
  • Extracted header that is commonly used by brands, creatives and sectors pages into a separate UI componenet HeaderUI

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.

1 participant