Implementing day view in calendar of event screen for both admin and user portal

[karthxk07]

What kind of change does this PR introduce?

Feature , Implements day view in calendar of event screen for both admin and user portal.

Issue Number:

Fixes #1422

Did you add tests for your changes?

No. No new file added, tests already present.

Snapshots/Videos:

Screencast.from.2024-01-18.20-39-24.webm

If relevant, did you update the documentation?

No, no documentation needed.

Summary

To fulfill the need for a multi-calendar view in both the admin and user portals, necessitating a solution to enhance the calendar functionality.
Implemented a day view for the calendar, similar to the layout observed in Google Calendar. This would involve displaying events on a daily basis for improved user experience and functionality.

Does this PR introduce a breaking change?

No, this PR does not introduce breaking changes

Have you read the contributing guide?
Yes

[github-actions]

Our Pull Request Approval Process

We have these basic policies to make the approval process smoother for our volunteer team.

Testing Your Code

Please make sure your code passes all tests. Our test code coverage system will fail if these conditions occur:

1. The overall code coverage drops below the target threshold of the repository
2. Any file in the pull request has code coverage levels below the repository threshold
3. Merge conflicts

The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing.

Reviewers

When your PR has been assigned reviewers contact them to get your code reviewed and approved via:

1. comments in this PR or
2. our slack channel

Reviewing Your Code

Your reviewer(s) will have the following roles:

1. arbitrators of future discussions with other contributors about the validity of your changes
2. point of contact for evaluating the validity of your work
3. person who verifies matching issues by others that should be closed.
4. person who gives general guidance in fixing your tests

CONTRIBUTING.md

Read our CONTRIBUTING.md file. Most importantly:

1. PRs with issues not assigned to you will be closed by the reviewer
2. Fix the first comment in the PR so that each issue listed automatically closes

Other

1. 🎯 Please be considerate of our volunteers' time. Contacting the person who assigned the reviewers is not advised unless they ask for your input. Do not @ the person who did the assignment otherwise.
2. Read the CONTRIBUTING.md file make

[aashimawadhwa]

can you change the calendar lines to gray that is used in the code and not black? and how does it looks in the mobile view? @karthxk07

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c756a73) 96.23% compared to head (b721198) 96.49%.
Report is 7 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1447      +/-   ##
===========================================
+ Coverage    96.23%   96.49%   +0.25%     
===========================================
  Files          133      134       +1     
  Lines         3404     3511     +107     
  Branches      1031     1090      +59     
===========================================
+ Hits          3276     3388     +112     
+ Misses         123      118       -5     
  Partials         5        5              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[karthxk07]

@aashimawadhwa I have changed the borders to gray and pushed the commit

Screencast.from.2024-01-19.16-55-02.webm

[aashimawadhwa]

I have few more remarks for the UI changes, if you could implement those too?

• change the color of the time as well, please understand we have not used black color in admin that much.
• reduce the spacing between the rows
• add am and pm instead of using 24 hrs format
• reduce the size of font used for time
• show a bar to indicate the current time

Note: please try to replicate design from goggle calendar. we are really working on fine details for UI along with functionalities.
@karthxk07

[karthxk07]

@aashimawadhwa Ok, I will be working on that

[noman2002]

@karthxk07 Please write the tests for the lines you have added, its dropping significantly.

[karthxk07]

@noman2002 ok

[noman2002]

Please write the test for the lines you have added and get the coverage of modified lines to 100%.

[karthxk07]

@noman2002 i am almost done with the changes , i just need to add all the tests. I will push the commits

[karthxk07]

@aashimawadhwa I am done adding the requested changes. Does this look fine?

Screencast.from.2024-01-28.12-19-48.webm

@noman2002 I will start working on adding the test for uncovered lines now

[karthxk07]

@noman2002 I have completed adding the tests too , ☺️ can I get the pull request reviewed please ?

[palisadoes]

Please fix the failing tests

1. Introspection

[karthxk07]

@palisadoes working on it sir 👍

Please fix the failing tests

1. Introspection

[palisadoes]

Please fix the failing tests

• Code coverage is declining. The tests for the files you have submitted are not complete

[karthxk07]

@palisadoes Sir added tests for uncovered lines

[palisadoes]

Please fix the introspection error. New API code was pushed and Admin needs to be in sync.

[karthxk07]

@palisadoes Sir , I am not sure why the error is occurring. The new graphql schema does not contain any requirement of UserToReturn type, which is the cause for the error. The new commits to the api repo have fixed this but i am not sure why this error is still occurring. On my local system the files validate properly.

[palisadoes]

Reimport the API data

[noman2002]

Looks fine to me.