cms icon indicating copy to clipboard operation
cms copied to clipboard

FEAT: Add calendar, Fixed Lint errors

Open vanshavenger opened this issue 1 year ago • 17 comments

In this i have added a calendar component for the Cohort students and admin can access and update events, so that students can just come and check on which day which topic in class happened and find its video link and Notes attached. And Students can see upcoming agenda at the single place that is updated by the admin of the next entire 30 days from the current date.

Attaching images of the added page, And proof of no lint errors.

Screenshot 2024-08-16 at 3 24 57 PM Screenshot 2024-08-16 at 3 24 45 PM Screenshot 2024-08-16 at 3 24 38 PM Screenshot 2024-08-16 at 3 24 29 PM Screenshot 2024-08-16 at 3 26 25 PM

Checklist before requesting a review

  • [x] I have performed a self-review of my code
  • [x] I assure there is no similar/duplicate pull request regarding same issue

vanshavenger avatar Aug 16 '24 09:08 vanshavenger

Nice addition

raashid16 avatar Aug 16 '24 10:08 raashid16

@devsargam @siinghd

hkirat avatar Aug 16 '24 16:08 hkirat

If it gets merged, /bounty $100

hkirat avatar Aug 16 '24 16:08 hkirat

Reviewing

devsargam avatar Aug 16 '24 16:08 devsargam

@devsargam fixed all the changes you asked for, kindly check. The calendar script was there as that can be used for production use case also, as it wil fill all blocks of friday, sat,sunday till May, and the admin then just have to update them or delete them

vanshavenger avatar Aug 17 '24 07:08 vanshavenger

I will review again today evening 🎉

devsargam avatar Aug 17 '24 07:08 devsargam

@vanshavenger fix lock please.

siinghd avatar Aug 18 '24 21:08 siinghd

@siinghd the yarn is identical completely, I again copied it from the main and it is showing no difference when i try to commit again for the verification.

vanshavenger avatar Aug 18 '24 21:08 vanshavenger

@vanshavenger sir look at package-lock and pnpm lock

siinghd avatar Aug 18 '24 21:08 siinghd

@siinghd donee serr

vanshavenger avatar Aug 18 '24 21:08 vanshavenger

@devsargam @siinghd resolved the merge conflicts.

vanshavenger avatar Aug 20 '24 07:08 vanshavenger

Checking this

devsargam avatar Aug 20 '24 07:08 devsargam

Week tab doesn't work on the calandar image

devsargam avatar Aug 20 '24 08:08 devsargam

Maybe we could just have month + agenda tab. Week and day are just subsets of them.

devsargam avatar Aug 20 '24 08:08 devsargam

@hkirat thoughts?

devsargam avatar Aug 20 '24 08:08 devsargam

Screenshot 2024-08-20 at 1 41 50 PM

@devsargam the week is showing , i just ran the seed script and it is showing the entries. and da and week come under react-big-calendar only.

vanshavenger avatar Aug 20 '24 08:08 vanshavenger

@devsargam @hkirat if any further changes are required let me know, happy to resolve them asap :))

vanshavenger avatar Aug 20 '24 08:08 vanshavenger

/bounty $100

hkirat avatar Aug 24 '24 16:08 hkirat