FossifyOrg / Notes

A simple text field for adding quick notes without ads.
https://www.fossify.org
GNU General Public License v3.0
158 stars 15 forks source link

Password protect the whole aplication and notes deletion #20

Closed WillBBHM closed 5 months ago

WillBBHM commented 5 months ago

What is it?

This is the link of the feature request : https://github.com/FossifyOrg/Notes/issues/17

Description of the changes and new features

Screenshots

Acknowledgement

Aga-C commented 5 months ago

Why haven't you did it similarly as it's done in other apps? Why new strings, new config items?

Check how it's done in Gallery:

BTW. In template there was Fixes # because GitHub will automatically close original issue after merging if this syntax is used. It makes our work easier, because we don't need to manually close issues then.

WillBBHM commented 5 months ago

The new configuration items and character strings were added because they weren't already there (as they're not a note but the application, I had to modify them slightly). As for the config, it's because the application remembers its passwords, but the application itself isn't a note, and I wanted the application to take the same form as the way notes are protected, to protect the application.

Aga-C commented 5 months ago

I know they weren't defined in Notes, but they are already in Commons, which is imported everywhere, and you can just use it. Look at the Gallery code I sent you, there's also a whole app blocked, and it's fully using things from Commons.

WillBBHM commented 5 months ago

As I'm still a computer science student, could you please help me implement it? Or at least guide me through it? With common. I'm going to try and implement it myself by looking at other applications.

Aga-C commented 5 months ago

Check links I've sent you a few comments before. They point exact lines of code.

naveensingh commented 5 months ago

I'm still a computer science student, could you please help me implement it?

Thanks for your contribution but no. As Agni said above, check how it's done in other Fossify apps.

We won't be "helping" people implement anything as that will invite more and more people who are beginning to learn Android development and helping people learn development is not the mission.