kiwix-js
kiwix-js copied to clipboard
Remove unused functions from util.js (possibly also elsewhere)
Quite a few functions exported in util.js
are unused currently in the app. Some may be interesting techniques that we want to keep for reference, but then we shouldn't expose them. Others are obsolete and should be removed.
Any PR based on this issue should await merging of #838 because some functions are already being removed there.
#838 has been merged, it should be easy (my guess) to implememt that one now.
Hello there! I want to resolve this issue. Can someone please guide me for that ?
Hello!
Just to let you know that I am working on this issue given that no PR has been submitted since January.
@Bam92 hey there mate, I would like to confirm if you still working on this issue?
Hey @dipenr06,
Yes, I planned to work on in this hacktoberfest.
Don't mind if I do?
can I work on this issue?
@Bam92 It appears you didn't get a chance to work on this in October, so unless I hear from you, I'll assign it to the next person in the queue who has expressed an interest, @dheerajdlalwani. @amitpanwar789 I'd suggest you choose something else unless @dheerajdlalwani says he's not interested.
I have exams going on right now. It will be some time before I can get to it. So in the mean time, i think @amitpanwar789 can go for it?
OK, thank you for letting me know @dheerajdlalwani. I'll wait a little to hear from @Bam92, and if he is no longer interested, I'll assign to @amitpanwar789.
@Jaifroid Should I wait or start working on it?
@amitpanwar789 You can go ahead, thanks!
@Jaifroid should I remove those unused functions completely from the util.js or just remove them only from return(export)?
@amitpanwar789 You should remove them completely, please. They will remain in the git history in case anyone ever needs to find them again.