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

feat: Support approve/reject all button: #502

Merged
merged 23 commits into from
Nov 3, 2024

Conversation

ghost
Copy link

@ghost ghost commented Aug 26, 2024

Description

Increase the pagination count in pending requests and display the cards as columns not rows and implement a new button for approving/rejecting all pending requests

Changes

  • Display the cards in cols instead of rows
  • Create an approve all btn to approve all vacation requests
  • Create reject all btn to reject all vacation requests

Related Issues

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstrings
  • Screenshots/Video attached (needed for UI changes)

- Display the cards in cols instead of rows
- Create an approve all btn to approve all vacation requests
- Create reject all btn to reject all vacation requests
@ghost ghost marked this pull request as draft August 26, 2024 08:38
@Mahmoud-Emad Mahmoud-Emad marked this pull request as ready for review August 26, 2024 14:31
@Mahmoud-Emad Mahmoud-Emad self-assigned this Aug 27, 2024
@zaelgohary
Copy link
Collaborator

Pagination is not updated when doing approve all or reject all.

screen-capture.1.webm

Copy link
Collaborator

@zaelgohary zaelgohary left a comment

Choose a reason for hiding this comment

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

Request status is still pending in Calendar although I clicked approve all.

image

@zaelgohary
Copy link
Collaborator

Request status is still pending in Calendar although I clicked approve all.

Not yet resolved.

image

image

@Mahmoud-Emad Mahmoud-Emad force-pushed the development_approve_all_pending_requests branch from 25a9ea3 to 05154d2 Compare October 21, 2024 10:15
@zaelgohary
Copy link
Collaborator

Issue is still not fixed. Request is being shown as pending while it's approved in the network tab.

image

@zaelgohary
Copy link
Collaborator

Also, the notifications are messed up as the notification that should appear to the supervisor, gets shown to the user.

image

Copy link
Collaborator

@zaelgohary zaelgohary left a comment

Choose a reason for hiding this comment

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

Issue is still not fixed. The request is being shown as pending while it's approved in the network tab.
This is still not fixed.

image

The calendar is being updated from team member's side however, it doesn't when the user is the approval.

image

@Mahmoud-Emad Mahmoud-Emad force-pushed the development_approve_all_pending_requests branch from 63a6a2e to 26c76b0 Compare October 29, 2024 10:49
- Fix the 'unique_together' issue.
- Fix approving all vacations.
- Fix updating the status of the vacations.
- Update the vacation balance after the user do the action.
Copy link
Collaborator

@zaelgohary zaelgohary left a comment

Choose a reason for hiding this comment

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

LGTM

@Mahmoud-Emad Mahmoud-Emad merged commit a4dea72 into development Nov 3, 2024
@Mahmoud-Emad Mahmoud-Emad deleted the development_approve_all_pending_requests branch November 3, 2024 14:33
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.

2 participants