Skip to content

Conversation

@JothishKamal
Copy link
Collaborator

@JothishKamal JothishKamal commented Aug 11, 2025

Pull Request Type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Purpose

  • WHAT is this PR for?
  • Fixes course title not being set when creating a reminder
  • Fatal Exception: java.lang.IllegalArgumentException
    Key "0Timestamp(seconds=1754904600, nanoseconds=0)_BCSE334L_F2_Timestamp(seconds=1754907600, nanoseconds=0)" was already used. If you are using LazyColumn/Row please make sure you provide a unique key for each item.

Approach

  • HOW does this change fulfill the purpose?
  • Fixes the bug by setting course ID and title when adding a reminder in AcademicsScreenContent
  • Fixes the bug by generating a random UUID for each period

Does this PR introduce any new bug?

  • Yes
  • No
  • Maybe

Issue Reference: #63

@JothishKamal JothishKamal requested a review from iamyajat as a code owner August 11, 2025 09:24
@JothishKamal JothishKamal changed the title fix: course title not set when creating a reminder fix: course title not set when creating a reminder + unique key for timetable Aug 11, 2025
@dk-a-dev
Copy link
Contributor

yep this should work

@dk-a-dev dk-a-dev merged commit 96f22c7 into GDGVIT:v3 Aug 13, 2025
2 checks passed
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