这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Mastermind-sap
Copy link
Contributor

@Mastermind-sap Mastermind-sap commented Mar 27, 2025

Description

  • Added task swipe tour.
  • Also fixed certain previous versions of packages that were causing error building.

Fixes #459

Screenshots

image

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

@Mastermind-sap Mastermind-sap changed the title Added task swipe tour feat : added task swipe tour Mar 31, 2025
@Pavel401
Copy link
Member

@SGI-CAPP-AT2 Please review this when you get time.

@SGI-CAPP-AT2 SGI-CAPP-AT2 self-requested a review May 26, 2025 14:47
Copy link
Collaborator

@SGI-CAPP-AT2 SGI-CAPP-AT2 left a comment

Choose a reason for hiding this comment

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

Please remove generated or unintended files.
PR will be gtg then.

import path_provider_foundation
import shared_preferences_foundation
import sqflite_darwin
import sqflite
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Mastermind-sap are these changes intended? if not, we should discard it.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this

@Mastermind-sap
Copy link
Contributor Author

Hey @SGI-CAPP-AT2 ,
Those changes occur when we compile for macOS. Those are intended and should not cause problems in future.

If you still want you can ignore those generated files.

Copy link
Collaborator

@SGI-CAPP-AT2 SGI-CAPP-AT2 left a comment

Choose a reason for hiding this comment

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

Tested and runs fine !

@Mastermind-sap
Copy link
Contributor Author

@SGI-CAPP-AT2 I have updated the gitignore and removed the generated files.

@SGI-CAPP-AT2 SGI-CAPP-AT2 self-requested a review May 29, 2025 01:13
Copy link
Collaborator

@SGI-CAPP-AT2 SGI-CAPP-AT2 left a comment

Choose a reason for hiding this comment

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

Hey Sorry for misunderstanding,
I wanted the files to be just discarded from that commit and not to be ignored using gitignore.
Do not add files to .gitignore instead do not stage them for commit manually
We should discuss about .gitignore changes in another issue.

@Mastermind-sap
Copy link
Contributor Author

@SGI-CAPP-AT2 I have removed the previous commit like you asked.

Copy link
Collaborator

@SGI-CAPP-AT2 SGI-CAPP-AT2 left a comment

Choose a reason for hiding this comment

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

Please remove those generated files

Copy link
Collaborator

Choose a reason for hiding this comment

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

please remove this as well from commit

import path_provider_foundation
import shared_preferences_foundation
import sqflite_darwin
import sqflite
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this

@Mastermind-sap
Copy link
Contributor Author

I cannot understand why I need to do this. Anyways I have done as you said @SGI-CAPP-AT2 .

@SGI-CAPP-AT2
Copy link
Collaborator

I cannot understand why I need to do this. Anyways I have done as you said @SGI-CAPP-AT2 .

Just to keep PR and commits clean
You can see how much changes are decreased by discarding the lock file. (From 800+ to only 241)

@SGI-CAPP-AT2 SGI-CAPP-AT2 merged commit 4327d71 into CCExtractor:main Jul 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Task Swipe Tutorial

3 participants