Skip to content

feat: Add Pseudo Logcat Reader#257

Merged
hasrat-ali merged 6 commits intoSketchware-Pro:mainfrom
khaled-0:por-LogCat
May 27, 2022
Merged

feat: Add Pseudo Logcat Reader#257
hasrat-ali merged 6 commits intoSketchware-Pro:mainfrom
khaled-0:por-LogCat

Conversation

@khaled-0
Copy link
Contributor

@khaled-0 khaled-0 commented Oct 14, 2021

This PR adds a full fledged Logcat Reader and Functionality for All(Hopefully) Log Related Stuffs in The Projects Made With.
To enable logcat support in your apps, enable Logcat Option from build settings.
Make sure you don't keep it enabled in the final builds unless you know what you're doing

Function Wise (if Logcat Enabled)
Made projects will include a class SketchLogger.java
This Class Uses java.lang.Process to execute and read each and every (hopefully) logs from the app itself. Then using a BuffereredReader, it Broadcasts the logs one by one with an intent filter RECEIVE_NUB_LOGS which than gets picked by Sketchware.
Also in order for that SketchLogger.java class to work one must call SketchLogger.startLogging(Context) from somewhere in the app. It's usually done automatically in SketchApplication.java but if for some reason user doesn't use the default application class they must use that method in order for the logcat to work.
More info can be found in the class itself*

As for Reading The broadcast , Open the Logcat Reader from Design Menu in Sketchware Pro before opening the application. So you won't miss out the logs of app startup.

Finally. I don't know what else to write anymore.🙏🙏

j-px
j-px previously requested changes Oct 14, 2021
@j-px
Copy link
Contributor

j-px commented Oct 14, 2021

Why's this PR marked as WIP btw?

@j-px j-px added the feature A new feature label Oct 14, 2021
@khaled-0 khaled-0 changed the title Por log cat feat: Adding Almost Every Logcat Reader Features Oct 14, 2021
@khaled-0 khaled-0 marked this pull request as ready for review October 14, 2021 17:30
@khaled-0
Copy link
Contributor Author

Why's this PR marked as WIP btw?

Was writing description. Its open now

@khaled-0 khaled-0 requested a review from j-px October 15, 2021 02:53
@khaled-0
Copy link
Contributor Author

i haven't checked the updated DesignDrawer while resolving conflicts. please kindly test that out when you can, should work tho

@khaled-0
Copy link
Contributor Author

These commits are a mess ngl 😔

khaled-0 and others added 5 commits May 15, 2022 01:13
renamed and removed some ids

feat: Adding Logcat Reader Support

LogReaderActivity receives and shows Log broadcasts, processes and shows them in a recyclerview.

Added configChanges in Manifest for Different Screen sizes
Changed To ``ScrollToPosition`` instead of ``..WithOffset``

Remove All XML References and Make Views Programmatically

Veri easy

Update LogReaderActivity.java

Update LogReaderActivity.java

Added Layout Tags And Fixed LayoutParams

fix por moments
Add more stuffs for creating project with logger enabled

Update yq.java
For Quick Access :thenking:
from ``RECEIVE_NUB_LOGS`` to com.sketchware.remod.ACTION_NEW_DEBUG_LOG

Update LogReaderActivity.java

followup of previous commit

fix compilation error caused by R
@khaled-0
Copy link
Contributor Author

cleaned most of messy commits as much as I could.

@khaled-0 khaled-0 changed the title feat: Adding Almost Every Logcat Reader Features feat: Add Pseudo Logcat Reader May 14, 2022
@hasrat-ali hasrat-ali merged commit 6a3a9a1 into Sketchware-Pro:main May 27, 2022
@khaled-0 khaled-0 deleted the por-LogCat branch June 6, 2022 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants