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

Blink iOS creates an always-empty snips folder in On My iPhone local storage, even when snips storage is set to iCloud on a fresh install (re-open #2061) #2062

Open
3 tasks done
jimmy-zhening-luo opened this issue Oct 22, 2024 · 4 comments
Labels
bug Bug that's either related to Blink Shell or prevents the user to use the app in normal conditions.

Comments

@jimmy-zhening-luo
Copy link

Checklist

Configuration

Version

  • Blink v18.0.1.940
  • iOS 18.0.1

Settings

In-App
  • iCloud Sync: On
  • Snips →
    • Default Location: iCloud Drive
    • Blink Snips Index: Off
  • X Callback URL: On
iCloud
  • Blink: On
  • iCloud Drive →
    • Blink: On
iPhone
  • Blink →
    • Document Storage: iCloud Drive

Describe the bug

Repro Steps

  1. Install Blink on a fresh iPhone. (If needed, configure Blink according to settings above)
  2. Launch Blink and create a snip.
  3. Launch Files app and navigate to iCloud FilesBlinksnips. Verify the presence of your newly created snip.
  4. In Files app, navigate to On My iPhone.

Expected Behavior

Local Blink folder is hidden because there's nothing in it.

Actual Behavior

Local Blink folder is visible, because there's a single empty folder in there called snips, which always contains 0 snips.

If the snips folder is deleted, Blink automatically creates a new one on next launch.

Screenshots

✅ iCloud Drive

iCloud Drive: correct behavior

❌ On My iPhone

On My iPhone: wrong behavior

@jimmy-zhening-luo jimmy-zhening-luo added the bug Bug that's either related to Blink Shell or prevents the user to use the app in normal conditions. label Oct 22, 2024
@jimmy-zhening-luo
Copy link
Author

@carloscabanero, you closed my bug report #2061 without verifying resolution. I'll remind you that I am a paying customer of Blink Plus for years now, nearly since you began offering the service.

Here are my follow-up questions about this BUG:

  1. I am aware of dotfiles and how they work, and there are no dotfiles in there, as confirmed using Textastic and as confirmed by my ability to make the folder disappear by deleting the snips folder, and its reappearance only occurring on re-launching Blink. You would know this if you read my detailed bug. My question is: Why would there be a .github folder generated if I have use snips library turned OFF?

  2. Your answer would explain why there would be a snips folder on my chosen file storage location, but my file storage location is set to iCloud. See my detailed configuration above to note that I have it set to iCloud in all three places possible. If you are correct, then why is Blink creating a .github folder for a feature (snips library) that I turned OFF, in a storage location that is the opposite of the place I specified?

@carloscabanero
Copy link
Member

carloscabanero commented Oct 22, 2024

Hi Jimmy, sorry if I closed the bug too soon, and don't take it personally. I usually do that when I think there is no malfunction and that the issue is closer to a support request. I read the Issue and in this case you are right, but I actually forgot about one reason why we made that decision back in the day.

You can see the logic that creates the folders here (advantages of being open source):

if !fm.fileExists(atPath: snippetsLocation.path()) {

As you can see, we always create the snips local folder, but we don't initialize the cache if we are not asked to do so, etc... And your question is on point, should we then skip initializing the Snips folder as well?

There are two reasons why back in the day (if I recall), we decided to keep it:

  1. We thought people may still want to play with the local directory, maybe copying some of the Snips there if they wanted to keep them private, etc... That's also the reason why the folder is called "snips" instead of ".snips", as we understood the user may want to keep it within its reach.
  2. What you mention as the "Expected behavior": Local Blink folder is hidden because there's nothing in it. Funny as it is, we had many questions back in the day because users couldn't find that folder. You need to touch some file or move something there from the shell to be displayed and that was creating confusion. Because for a lot of people the "Expected Behavior" was that this folder had to be initialized with Blink, we decided this could kill two birds with one stone. The alternative would have been to create a README or some sort of file there. But needing 1, we discarded this.

So answering your other questions:

  1. Why would there be a .github folder generated if I have use snips library turned OFF?
  • This was incorrect from my side. We create the snips folder but the .cache/github if you have it off.
  1. If you are correct, then why is Blink creating a .github folder for a feature (snips library) that I turned OFF, in a storage location that is the opposite of the place I specified?
  • I was not correct. We decided to create it because of Snips operation and because of the Expected Behavior with that folder.

Now, that out of the way, and that this is our "Expected Behavior", I understand you would prefer to get rid of it?

@carloscabanero
Copy link
Member

Hi! Were you able to take a look at this?

@carloscabanero
Copy link
Member

Please let me know if you consider there may be other changes necessary. Otherwise it works per our specifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug that's either related to Blink Shell or prevents the user to use the app in normal conditions.
Projects
None yet
Development

No branches or pull requests

2 participants