TrueSparrowSystems / AI-SalesSparrow-Android-App

MIT License
1 stars 0 forks source link

added delete task from account details screen #81

Closed dakshbhardwaj closed 1 year ago

dakshbhardwaj commented 1 year ago

Summary:

coderabbitai[bot] commented 1 year ago

Walkthrough

This pull request introduces changes to the API service, common components, and screens in the sales application. It modifies endpoint URLs, enhances UI components with additional parameters for better customization, and introduces a new function for task deletion. Debugging logs have been added and certain code blocks have been commented out for further investigation.

Changes

File Summary
.../sales/api/ApiService.kt Modified the endpoint URL from absolute to relative path in the @DELETE annotation.
.../sales/common_components/EditableTextField.kt Removed modifier semantics and replaced it with a nullable modifier parameter.
.../sales/common_components/TaskSuggestionCard.kt Added new parameters shouldShowOptions and onCancelTaskClick, and conditional rendering of menu options.
.../sales/screens/NotesScreen.kt Commented out a block of code, added debugging logs, modified an if condition, and added a new parameter to the TaskCard function.
.../sales/viewmodals/GlobalStateViewModel.kt Introduced a new function DeleteTaskById for task deletion.

🐇💻

"In the land of code, where logic is king,

A rabbit hops by, making changes with a zing.

From APIs to UI, across screens wide,

With every hop, improvements abide.

Debugging logs here, a function there,

Making the code cleaner, beyond compare.

So let's celebrate these changes, oh so bright,

For they bring to our code, a new light." 🎉🌟

Tips ### Chat with CodeRabbit Bot (`@coderabbitai`) - Mention `@coderabbitai` in any *review comment* for bot assistance. The bot stays silent unless called upon. - Note: Review comments are made on code diffs or files, not on the PR overview. ### Pause Incremental Reviews - Insert `@coderabbitai: ignore` in the PR description to halt the bot's ongoing reviews. Remove the line to resume.