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

Technical debt issue13 #3

Closed
wants to merge 2 commits into from
Closed

Conversation

gs205436
Copy link
Collaborator

@gs205436 gs205436 commented Feb 8, 2024

Thanks for your contribution.

PLEASE REMOVE

To support us in providing a nice (and fast) open-source experience:

  1. Verify that the tests are passing
  2. Check that the code is properly formatted (using AndroidStudio's autoformatter)
  3. Provide write access to the branch
  4. If the PR is not ready for review, please submit it as a draft

PLEASE REMOVE

Describe the pull request
A clear and concise description of what the pull request changes/adds.

Link to the the issue
(If available): The link to the issue that this pull request solves.

License agreement
By opening this pull request, you are providing your contribution under the Apache License 2.0 (see LICENSE.md).

Note: new dependencies/libraries
Please refrain from introducing new libraries without consulting the team.

@gs205436 gs205436 added the Technical Debt Technical Debt label Feb 8, 2024
@gs205436 gs205436 requested a review from thanhpd February 8, 2024 21:47
@gs205436 gs205436 self-assigned this Feb 8, 2024
@thanhpd thanhpd closed this Feb 8, 2024
Reva-B98 added a commit that referenced this pull request Feb 17, 2024
…ply with Java Language Specification. Changed position from private final static to private static final in accordance with Java Language Specification
Sharanyu pushed a commit that referenced this pull request Feb 17, 2024
Changed the redundant string to variable.
Soham-2411 pushed a commit that referenced this pull request Feb 17, 2024
…neric #3

Thrown a dedicated exception for TrackFileFormat of uri instead of generic #3
arextron pushed a commit that referenced this pull request Apr 9, 2024
…nounce-max-slope-at-end-of-each-recording

97 settings voice announcements announce max slope at end of each recording.
nihal514 added a commit that referenced this pull request Apr 9, 2024
gup-abhi added a commit that referenced this pull request Apr 9, 2024
thanhpd added a commit that referenced this pull request Apr 9, 2024
1. Updated string.xml to fix error in screenshot #1 (src\main\res\values\strings.xml)
2. Updated build.gradle to fix error in screenshot #2 (src\main\res\layout\fragment_runs_stat.xml)
3. Updated src\main\java\de\dennisguse\opentracks\stats\TrackStatisticsUpdater.java to fix error in screenshot #3
4. Commented out code in src/main/java/de/dennisguse/opentracks/ui/aggregatedStatistics/SeasonStats/RunAndChairStatActivity.java and src/main/java/de/dennisguse/opentracks/ui/aggregatedStatistics/SeasonStats/SeasonStatActivity.java to fix error in screenshot #4
5. Commented out code in  src/main/java/de/dennisguse/opentracks/ui/aggregatedStatistics/AggregatedStatisticsAdapter.java to fix error in screenshot #5
VishalPeruma1 pushed a commit that referenced this pull request Apr 26, 2024
fix: issue #3. add private constructor for Mapping class.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Technical Debt Technical Debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants