Skip to content

refactor, fix and optimize about activites and fragments#1682

Merged
hasrat-ali merged 1 commit intoSketchware-Pro:mainfrom
yoann256:refactor/about-activity-fragments
Jun 29, 2025
Merged

refactor, fix and optimize about activites and fragments#1682
hasrat-ali merged 1 commit intoSketchware-Pro:mainfrom
yoann256:refactor/about-activity-fragments

Conversation

@yoann256
Copy link
Contributor

  • AboutActivity.java got some fixes and optimizations
  • BetaChangesFragment.java and ChangeLogFragment.java got fixes and optimization

@trindadedev13 trindadedev13 added improvement Enhances an existing feature or internal logic. fix This pull request contains a bugfix labels Jun 24, 2025
@hasrat-ali hasrat-ali force-pushed the refactor/about-activity-fragments branch 2 times, most recently from ee8c624 to 1bab63d Compare June 29, 2025 19:55
Infinite scroll no longer calls loadCommitData() multiple times
Pagination no longer overwrites data
try-catch no longer outside network callback
@hasrat-ali hasrat-ali force-pushed the refactor/about-activity-fragments branch from 1bab63d to d555b41 Compare June 29, 2025 20:00
@hasrat-ali
Copy link
Contributor

After removing the useless changes, the changes that are left don't seem right to me.

And I also found a bug, it is not showing the current version, it is just showing new update.

But if you feel that we have made a mistake, please open a new pull request and explain the changes with full details.

@hasrat-ali hasrat-ali closed this Jun 29, 2025
@Edward-NewGate-KOC
Copy link
Collaborator

After removing the useless changes, the changes that are left don't seem right to me.

And I also found a bug, it is not showing the current version, it is just showing new update.

But if you feel that we have made a mistake, please open a new pull request and explain the changes with full details.

It's not his bug. The issue is simply because the commit hash doesn't exist in the main branch, which is why it appears that way
If you push his changes to the main branch, it will work fine

Btw, I reviewed his changes, and I believe they're unnecessary anyway

@Edward-NewGate-KOC
Copy link
Collaborator

Edward-NewGate-KOC commented Jun 29, 2025

Oh, you reverted some of his changes. I just noticed that

I think you should merge it, as his changes will prevent the app from crashing if GitHub API temporarily blocks the user's IP due to too many requests

@hasrat-ali hasrat-ali merged commit d555b41 into Sketchware-Pro:main Jun 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix This pull request contains a bugfix improvement Enhances an existing feature or internal logic.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants