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.

Sign up for GitHub

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

Jump to bottom

Feature/exchange #281

Draft
wants to merge 61 commits into
base: develop
Choose a base branch
from
Draft

Feature/exchange #281

wants to merge 61 commits into from

Conversation

tomaspalma
Copy link
Member

@tomaspalma tomaspalma commented Sep 5, 2024 •
edited
Loading

A todo list:

  • Preview schedule on exchange cards
  • Make the filters "Todos os pedidos", "Meus pedidos", "Recebidos"
  • Show classes on each UC when the exchange card is hovered on
  • Improve logic of submitting both direct and exchange requests
  • Make the exchange requests view filters about ucs work
  • Make the exchange requests view filters about the classes inside of the request work
  • Make the "Propôr troca" button working on the request cards

@netlify Netlify
Copy link

netlify bot commented Sep 5, 2024 •
edited
Loading

❌ Deploy Preview for tts-fe-preview failed.

Name Link
🔨 Latest commit 5988f51
🔍 Latest deploy log https://app.netlify.com/sites/tts-fe-preview/deploys/6739da5cb8f21100088e2d6b

@tomaspalma tomaspalma force-pushed the feature/exchange branch 4 times, most recently from f177ce6 to 634f30a Compare September 12, 2024 09:52
tomaspalma and others added 13 commits September 13, 2024 18:19
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
chore: add route for feup exchange
df4d9b6 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
add: schedule and initial sidebar on exchange page
5592396 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
add: static frontend to see and create requests
c4539df 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
@tomaspalma
placeholder: started trying with a useSession hook
f311b78
@tomaspalma
add: csrf token retrieval & started authentication system
b369c8c
@tomaspalma
feat: exchange and planner page share states
b8efa14
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
add: list of class names as a base to the filters
7d38ce4 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
feat: added initial mechanism to send exchange requests
12e1deb 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
@tomaspalma
feat: initial version of using useswrinfinite to fetch exchanges pagi…
e2663dd 
…nated
@tomaspalma
refactor: decoupled logic of getting the classes from <Schedule>
a674013
@tomaspalma
feat: expand class information about request card ucs when hovered
d6cf99e
@tomaspalma
feat: pagination for the all requests view
1347741
@tomaspalma
feat: show schedule and dynamically show course units and request cards
96dfbfb
@tomaspalma @pfpo
@eduardooliveiraps @joaonevesf
feat: fetching request card metadata
2f36c9a 
Co-authored-by: pfpo <[email protected]>
Co-authored-by: eduardooliveiraps <[email protected]>
Co-authored-by: joaonevesf <[email protected]>
tomaspalma and others added 16 commits October 1, 2024 23:44
@tomaspalma
Merge branch 'feature/exchange' of github.com:NIAEFEUP/tts-fe into fe…
5c78ddd 
…ature/exchange
@jose-carlos-sousa
fix : now both classes have value of pratical one
839871c
@tomaspalma
Merge pull request #317 from NIAEFEUP/fix/exchangeSchedule
7eaa15a 
fix : now both classes have value of pratical one
@tomaspalma
feat: all course units selected on request card when card open
21ad51f
@tomaspalma
refactor: simplified logic of <CreateRequest> component
08b6f4e
@tomaspalma
feat: started exchange class filters
dc0d01a
@tomaspalma
feat: choose filters in frontend
aea47ee
@tomaspalma
feat: sending base64 of classes filter to backend
4b2a88a
@tomaspalma
feat: skeleton loading animation for requests and filters
d8c9a42
@tomaspalma
feat: fetching authenticated user profile image
c8fdb66
@tomaspalma
feat: submitting request in propor troca
1432906
@tomaspalma
fix: deselecting an option had it remaining on the schedule
864244c
@tomaspalma
fix: restoring previewed changed schedule when creating a request
a0f15dd
@tomaspalma
feat: preview destination class schedule when creating request
a8ba8ab
@tomaspalma
fix: exchange sidebar not extending beyond screen height
49a468f
@tomaspalma
feat: using only course units that have an eligible exchange active p…
d20dea1 
…eriod
tomaspalma and others added 13 commits October 19, 2024 18:20
@tomaspalma
Merge branch 'develop' into feature/exchange
dad86f5
@tomaspalma
refactor: created common request card with common state and methods
3cf615c
@tomaspalma
refactor: changed direct exchange type attributes and more modular code
2d923be
@tomaspalma
feat: previewing working for received requests
c0262d6
exchange schedule is now previewed when hovering through class options
2344165
@tomaspalma
refactor: common card header in request card
51dc929
@tomaspalma
fix: marketplace exchange request card does not show request status
1b0e37b
@Wallen-Ribeiro
Merge pull request #334 from NIAEFEUP/feature/preview-schedule
de5ea85
@tomaspalma
feat: receiving request pending motive from backend
f1ef7c6
@tomaspalma
fix: two independent useswr calls using the same key
7ad7a27
@tomaspalma
add: changed button colors and spinner on logout
8287498
@tomaspalma
feat: log out is now instantaneous
d55fab2
@tomaspalma
feat: sent request card history
5988f51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tomaspalma @jose-carlos-sousa @Wallen-Ribeiro