Skip to content

Santiago Jiménez solution - #10

Open
Naryalin wants to merge 13 commits into
bobtheogre:mainfrom
Naryalin:SantiagoJiménez-Solution

Hidden character warning

The head ref may contain hidden characters: "SantiagoJim\u00e9nez-Solution"
Open

Santiago Jiménez solution#10
Naryalin wants to merge 13 commits into
bobtheogre:mainfrom
Naryalin:SantiagoJiménez-Solution

Conversation

@Naryalin

Copy link
Copy Markdown

Technical Interview Solution

Description

This pull request includes my solution for the technical interview. Key features implemented:

  1. Router Setup:

    • Completed router configuration for page navigation.
    • Fixed index.js in the router folder.
    • Integrated navigation to /data which routes to DataTable.vue.
  2. Navigation Bar:

    • Added navigation to the Data page using a navbar.
    • Implemented in App.vue with router integration.
    • Utilized NavButton.vue for easy addition of new routes in the navbar.
  3. Data Table with Search:

    • Completed the data table view with a search function.
    • Utilized Vuex store and integrated a getAllData service to fetch data.
    • DataTable.vue organizes the data in a table with dynamic headers.
    • Added a search bar for filtering the table content.
  4. PWA and Service Worker:

    • Finished PWA setup and service worker implementation.

Additional Notes

  • There are some conflicts with eslint-plugin-prettier and @vue/eslint-config-prettier@6.0.0 dependencies that may appear while running npm i. For resolving these issues, please refer to the instructions provided in the README. Despite these conflicts, the application runs without any problems.

Naryalin added 13 commits May 14, 2024 18:55
Fix node js version error from Open SSL
Add .eslintrc.js for Vue lint compatibility, fix line ending error in ESLint rules, add missing bracket in routes/index.js. Rename view files for multi-word consistency and update routes accordingly.
Wrap NavButton as component, adjust alignment of nav inside app-bar
Now store uses the service getAllData to fetch and manage the state of users.
Now size and type of data are customizable
Add flag "--legacy-peer-deps" to npm install because of conflicts between eslint-plugin-prettier and vue/eslint-config-prettier. Keep this note until find a configuration where "npm install" and "npm serve" works as intended.
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