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

[FE] Refactor: #266 fe rename and reorganise files and folders structure #310

Merged

Conversation

JuanDNJ
Copy link
Collaborator

@JuanDNJ JuanDNJ commented Oct 10, 2024

[FE] Refactoring: api folder "src/api"

372479313-c3210aba-de1a-4cc2-9368-c43259493981

[FE] Refactoring: "src/store"

374266039-56bbea33-e384-4f3e-a7c9-7ed88aa75aa2

[FE] Refactoring: "src/componetnts/"

Before

before_refactor_components

After

refactor_components

  • And some bugs fixed

JuanDNJ and others added 16 commits October 2, 2024 11:01
Changes to be committed:
	renamed:    src/api/student/getResourceStudentWithId.ts -> src/api/student/getStudentDetailWithId.ts
	modified:   src/store/thunks/getDetailResourceStudentWithIdThunk.ts
Changes to be committed:
	modified:   src/store/thunks/getDetailResourceStudentWithIdThunk.ts
Changes to be committed:
	renamed:    src/api/index.ts -> src/api/getResourceWithAxios.ts
	modified:   src/api/student/getStudentDetailWithId.ts
…anize-files-and-folders

266-3 [FE] Refactor : Reorganize redux store folders and tests
The data returned by the call to /student/{studentId}/resume/projects has changed, before it received "project_name", now it is "name".
Changes to be committed:
	modified:   src/components/studentDetailCards/projectsSection/ProjectsCard.tsx

Changes not staged for commit:
	modified:   src/interfaces/interfaces.ts
The data returned by the call to /student/{studentId}/resume/projects has changed, before it received "project_name", now it is "name".
Changes to be committed:
	modified:   src/interfaces/interfaces.ts
On line 40 the "name.slice()" method was used, which gave an error, so it was removed.
Changes to be committed:
	modified:   src/components/studentDetailCards/projectsSection/ProjectsCard.tsx
The data returned by the call to /student/{studentId}/resume/projects, in the end, was correct...
Now it works correctly, I have taken the opportunity to fix a conditional, which rendered the component incorrectly
Changes to be committed:
	modified:   src/components/studentDetailCards/projectsSection/ProjectsCard.tsx
	modified:   src/interfaces/interfaces.ts
Copy link
Collaborator

@tomi-casabona tomi-casabona left a comment

Choose a reason for hiding this comment

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

al fin va quedando

@tomi-casabona tomi-casabona merged commit 4361349 into main Oct 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[BE][FE] Fix: wrong column names in tables [FE] Rename and Reorganise files and folders structure
4 participants