Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vaibhav's Technical Debt #1 #1

Closed
Vaibhav2399 opened this issue Feb 15, 2024 · 1 comment
Closed

Vaibhav's Technical Debt #1 #1

Vaibhav2399 opened this issue Feb 15, 2024 · 1 comment
Assignees
Labels
Bug Something isn't working Group11
Milestone

Comments

@Vaibhav2399
Copy link
Owner

Vaibhav2399 commented Feb 15, 2024

Describe the bug
Need to cast one of the operands of this addition operation to a "double" in file opentracks.services.announcement.VoiceAnnouncementManager.java

If applicable:
image

Technical information

  • Device: [e.g. Nexus 7 (2013)]
  • OS: [e.g. LineageOS 15.1]
  • OpenTracks version: [e.g. v3.2.4]
  • OpenTracks commit id (only for nightly builds): [e.g. 4b333bb]
@Vaibhav2399 Vaibhav2399 added the Bug Something isn't working label Feb 15, 2024
@Vaibhav2399 Vaibhav2399 self-assigned this Feb 15, 2024
@Vaibhav2399 Vaibhav2399 changed the title Dummy Issue Vaibhav's technical debt #1 Feb 15, 2024
@Vaibhav2399 Vaibhav2399 changed the title Vaibhav's technical debt #1 Vaibhav's Technical Debt #1 Feb 15, 2024
Vaibhav2399 added a commit that referenced this issue Feb 16, 2024
@Vaibhav2399
Copy link
Owner Author

Issue has been solved in the latest merge

@Vaibhav2399 Vaibhav2399 added this to the Case Study 1 milestone Feb 17, 2024
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
#1 Replace Switch with If Statment
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Commit to fix the raised issue :

Rename local variable 'altitude_m' to comply with naming convention #1
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Issue #1: Override method hashCode() because class overrides equals()…
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Revert "Issue #1: Override method hashCode() because class overrides …
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Revert "#1 Replace Switch with If Statment"
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
#1 Replace this "switch" statement by "if" statements to increase readability.
Vaibhav2399 pushed a commit that referenced this issue Apr 6, 2024
This PR consists of changes for issues resolved by Group9.


[#1](tarekFerdous#1)
Define a constant instead of duplicating this literal "<open>1</open>" 3
times.

[#2](tarekFerdous#2)
Replace this usage of 'Stream.collect(Collectors.toList())' with
'Stream.toList()' Group 09

[#6](tarekFerdous#6)
Either re-interrupt this method or rethrow the "InterruptedException"
that can be caught here.

[#9](tarekFerdous#9)
Remove unused "TAG" private field.

[#11](tarekFerdous#11)
Use isEmpty() to check whether the collection is empty or not.

[#13](tarekFerdous#13)
Fixing technical debts for io/file/impoter/importService.java and
io/file/importer/importViewModel.java

[#14](tarekFerdous#14)
io/file/importer/KmlTrackImporter and io/file/importer/KmzTrackImporter

[#15](tarekFerdous#15)
Removing the Deprecated lines in the code Group(09)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Group11
Projects
None yet
Development

No branches or pull requests

1 participant