Skip to content

Conversation

@bootstraponline
Copy link
Contributor

No description provided.

Copy link
Contributor

@joehoag joehoag left a comment

Choose a reason for hiding this comment

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

Looks OK to me. (Though I am generally unfamiliar with this logic.)

Might we want to move the cloud_build_metrics folder to some more centralized location, like the mobile_shared repo?

@bootstraponline
Copy link
Contributor Author

Might we want to move the cloud_build_metrics folder to some more centralized location, like the mobile_shared repo?

I have thought about that. I'd want to discuss with the team first. This code doesn't really have an owner right now so I'm worried about moving it off repo. espresso in mobile-shared is actually used by all the Android apps.

@bootstraponline bootstraponline merged commit a003c1f into master Apr 15, 2019
@bootstraponline bootstraponline deleted the docs branch April 15, 2019 16:04
kristofnemere added a commit that referenced this pull request Nov 24, 2025
Fixed issue #11: Account-level calendar events were clickable when they
shouldn't be.

Changes:
- Added isClickable field to ToDoItemUiState (defaults to true)
- In ToDoListViewModel.mapToUiState(), detect account-level events by
  checking if contextType equals "Account" (case-insensitive)
- Set isClickable to false for account-level calendar events
- In ToDoListScreen.ToDoItemContent(), conditionally apply clickable
  modifier based on isClickable flag using Modifier.then()

Account-level calendar events now appear in the ToDo list but are
not clickable, preventing navigation to invalid URLs.

Test plan:
- Create account-level calendar events
- Verify they appear in ToDo list
- Verify clicking on them does nothing (no navigation)
- Verify other items (course/user events, assignments, etc.) remain clickable
- Verify checkbox and swipe-to-done still work on account events

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
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.

3 participants