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

Challenge front end tech test Developer Pablo Stolarczuk #18

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

Conversation

pablodavidst
Copy link

  1. First task: Addition of “Creatives” table in Library page
  • Creation of new repository folders.repository.

  • Creation of new domain folders.domain

  • Addition of new repository folders in repositories catalog (catalog/index)

  • Creation of new table CreativesTableWidget and development of the main requirement.

  • The format of the field "Uploaded date" is slightly adapted but displayed as"YYYY-MM-DD" because it is not defined explicitly the expected format and it can be easily changed later according to functional requirements.

  • I tried to reuse as much as possible the existent widgets or components but the ui component TableUI did not have the features I needed (for example, no headers are displayed or it is organized the data in 1 cell ) Finally I created a plain table using html for rendering the data of Creatives.

  • I could have created a reusable component providing a table having the features needed for the Creatives Table which could be reused in the rest of the proyect but I considered that it would take too much time in this moment and I have prioritized delivering the challenge as soon as possible.

  1. Second task: Addition of new Sectors Page
  • Creation of new repository sectors.repository where the functions getSectorsCount and getSectorsName are created simulating a query to the backend returning the corresonding arrays. coming from hardcoded json objects.

  • Creation of new domain sectors.domain. Here it is created and exported the interfaces that normally come from graphql schemas.

  • Addition of new repository sectors in repositories catalog (catalog/index)

  • Creation of new page "Sectors" (sectors.page) inside the "Business settings" context and development of the main requirement.

  • Addition of new route "/business-settings/sectors" inside business-settings.route and import of new component for rendering the new page Sectors.

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