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

Declare "lastInterval" on a separate line. #4

Closed
wants to merge 1 commit into from

Conversation

pranav687
Copy link
Collaborator

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.

@pranav687 pranav687 added the Technical Debt Technical Debt label Feb 8, 2024
@pranav687 pranav687 self-assigned this Feb 8, 2024
@thanhpd thanhpd marked this pull request as draft February 8, 2024 22:28
Copy link
Collaborator

@thanhpd thanhpd left a comment

Choose a reason for hiding this comment

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

Closing the PR as we should make the PR on the forked repo

@thanhpd thanhpd marked this pull request as ready for review February 8, 2024 22:29
@thanhpd thanhpd closed this Feb 8, 2024
@pranav687 pranav687 deleted the technicalDebtIssue11 branch February 11, 2024 18:26
@pranav687 pranav687 restored the technicalDebtIssue11 branch February 11, 2024 18:26
@pranav687 pranav687 deleted the technicalDebtIssue11 branch February 11, 2024 18:27
krispatel1001 pushed a commit that referenced this pull request Feb 14, 2024
Removal of Empty statements in ChartView.java
numanSlm added a commit that referenced this pull request Feb 16, 2024
Replace "replaceAll()" by a call to the "replace()" method.
Reva-B98 pushed a commit that referenced this pull request Feb 17, 2024
Reva-B98 pushed a commit that referenced this pull request Feb 17, 2024
Issue #4 is solved by using a more specific exception, such as Unsupp…
Sharanyu pushed a commit that referenced this pull request Feb 17, 2024
Soham-2411 pushed a commit that referenced this pull request Feb 17, 2024
nihal514 added a commit that referenced this pull request Apr 9, 2024
Added Shortest wait at chairlift Widget
gup-abhi pushed 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 added a commit that referenced this pull request Apr 26, 2024
VishalPeruma1 added a commit that referenced this pull request Apr 26, 2024
fix: issue #4, adding comment to empty 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.

None yet

2 participants