Skip to content

Grant Sorting#88

Merged
jarenadams21 merged 12 commits intomainfrom
jane
Feb 22, 2025
Merged

Grant Sorting#88
jarenadams21 merged 12 commits intomainfrom
jane

Conversation

@janekamata
Copy link
Contributor

ℹ️ Issue

Closes #44

📝 Description

Added sorting when a user clicks on the header

Briefly list the changes made to the code:

  1. Updated GrantItemProps to Date type for applicationDate
  2. Changed GrantLabels to handle on Click and track sort order
  3. Refactored GrantList to handle sorting

✔️ Verification

Testing sorting by clicking on headers

🏕️ (Optional) Future Work / Notes

Interaction with pagination needs to be fixed pending fix for pagination

@janekamata janekamata self-assigned this Feb 16, 2025
janekamata and others added 4 commits February 16, 2025 14:33
commit 725946d
Author: Ryaken Nakamoto <108379394+Ryaken-Nakamoto@users.noreply.github.com>
Date:   Wed Feb 12 09:11:12 2025 -0500

    Connecting Front-End to DynamoDB Grant Data (#72)

    * created satchel store for all grants fetched from the backend to display in grantlist

commit fa05bd1
Author: Jane Kamata <38163243+janekamata@users.noreply.github.com>
Date:   Wed Jan 29 19:02:58 2025 -0500

    ISO Date typed interface (#66)

    * Added navigation controls from account settings to grant list; changed dashboard to account

    * [JAN-1] Notifications base layer wrap-up

    * [JAN-11] Add navigation controls to go from the "dashboard" < -- > grant list

    * [JAN-1] Notifications base layer wrap-up

    * Created typed interface for ISO dates and tested

    ---------

    Co-authored-by: Jaren Adams <jarenadams@Jarens-MacBook-Air.local>

commit a91bc98
Merge: f888c3b b47a86d
Author: prooflesben <122566738+prooflesben@users.noreply.github.com>
Date:   Wed Jan 29 18:33:48 2025 -0500

    Merge pull request #67 from Code-4-Community/ryaken

    Fixed

commit b47a86d
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 29 18:12:57 2025 -0500

    got rid of dependency

commit 0bbf3b6
Author: Jaren Adams <jarenadams@Jarens-MacBook-Air.local>
Date:   Tue Jan 28 20:37:53 2025 -0500

    code style changes

commit 33b2948
Merge: 5f6d437 f888c3b
Author: Jaren Adams <jarenadams@Jarens-MacBook-Air.local>
Date:   Tue Jan 28 20:27:45 2025 -0500

    pull in recent updates

commit 5f6d437
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 20:01:04 2025 -0500

    go away lock

commit 4c858b5
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:58:46 2025 -0500

    cleared node modules in frontend

commit 53c37d0
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:49:30 2025 -0500

    fixed gitignore

commit 06fbf94
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:47:59 2025 -0500

    fix crypto error

commit a1c29aa
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:34:55 2025 -0500

    try4

commit bf1ed9a
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:29:16 2025 -0500

    try3

commit 48c1832
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 19:18:57 2025 -0500

    try2

commit 9b39987
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 22 18:58:54 2025 -0500

    json change

commit 52b5fcb
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Tue Jan 21 18:38:32 2025 -0500

    added save functionality status

commit c291cef
Author: ryaken-nakamoto <ryakenNakamoto@gmail.com>
Date:   Wed Jan 15 18:52:36 2025 -0500

    cleaning up App.css and index.css, margin removed on edges when grant is expanded
Copy link
Collaborator

@prooflesben prooflesben left a comment

Choose a reason for hiding this comment

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

lovely clean greatly commented code

@jarenadams21 jarenadams21 merged commit 8950c27 into main Feb 22, 2025
1 check passed
@janekamata janekamata deleted the jane branch March 30, 2025 19:19
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.

[JAN-6] Grant Filters

3 participants