Skip to content

anisette fixes#5

Open
amsam0 wants to merge 4 commits intonythepegasus:feature/anisette_settingsfrom
amsam0-org:feature/anisette_settings
Open

anisette fixes#5
amsam0 wants to merge 4 commits intonythepegasus:feature/anisette_settingsfrom
amsam0-org:feature/anisette_settings

Conversation

@amsam0
Copy link
Copy Markdown

@amsam0 amsam0 commented Jan 26, 2023

for some reason this got removed and I think it's why we can't use files app on your branch.

Signed-off-by: naturecodevoid <44983869+naturecodevoid@users.noreply.github.com>
commit 0815e06
Author: naturecodevoid <44983869+naturecodevoid@users.noreply.github.com>
Date:   Wed Jan 25 21:10:02 2023 -0800

    adi.pb fixes
@amsam0 amsam0 changed the title allow opening documents in files app again anisette fixes Jan 26, 2023
@amsam0
Copy link
Copy Markdown
Author

amsam0 commented Jan 26, 2023

I made some more fixes.

  • File not found error should be fixed (for some reason checking if it exists doesn't work correctly, so I switched to trying to get data and if the data is nil, it will get adi.pb. After writing to adi.pb it will try to get anisette data again by calling main again; maybe move this into another function to make sure nothing bad happens?)
  • Change DLOG to print
  • Removed the unused session parameter in fetchADIFile

Tested with ani.sidestore.io and my own outdated anisette server. I previously had issues with both new and outdated servers but now the issues are fixed

@Spidy123222
Copy link
Copy Markdown

I have not had any issues with this PR when testing.

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.

2 participants