Description

when uploading files, if you are uploading a file (same one or an other one) with the same name, it'll be stored.
The user must be advertised and specify his intention

Reproduction URL

https://github.com/ThomasSed/Storz

Reproduction steps

1. Go to "home"
2. Click on "upload"
3. upload a file
4. do the 3 first steps a second time, with the same file (for example) 
5. go to your repository, you must see 2 files with same name

Screenshots

![DESCRIPTION](LINK.png)

Logs

No response

Browsers

No response

OS

No response

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

🐛 function that should be able to catch errors and show the appropriate message properly

Basic Example

.

Drawbacks

.

Unresolved questions

No response

Add a filtering feature to the My Files page.

For example :

• Filter by File types ( .pdf/.png/.jpeg etc )
• Filter by File status ( public/private )

Description

everything is in the title.
Must only delete the selected file

Reproduction URL

https://github.com/ThomasSed/Storz

Reproduction steps

1. Go to repository
2. Click on a file
3. Click on the trash button
4. accept
5. Go back to repository

Screenshots

![DESCRIPTION](LINK.png)

Logs

No response

Browsers

No response

OS

No response

Setup a basic console/terminal messages/logs to show when HTTP requests are made and when the app is running. This way it helps with debugging especially during development.

When a request is sent: The request methods, endpoint and status code would be logged to the console. The standard logger would be used instead of console.log().

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

having a back to top function button at the bottom of the webpage to allow users to get back to top quickly

Basic Example

button that will return to top when clicked

Drawbacks

where to place it

Unresolved questions

No response

Do you need to update this alert for more good UX? I will update the button component below the text and give contrast only for delete button.

Inspiration:

Description

Currently, the application has set a width equal to 100%, even for mobile devices, which leads to content too attached to the margin of the page.

Reproduction URL

https://storz.pages.dev/app/myFiles/desc

Reproduction steps

1. Go to 'https://storz.pages.dev' (width < 975px)
2. Click on 'Sign in'
3. Enter a valid email
4. Go to Files section
5. Click on file name
6. Containers have no margin set

Screenshots

Logs

No response

Browsers

Chrome

OS

Windows

Add pagination in the table view of the My Files page.

Description

Currently, the application doesn't manage special characters like 'ó', 'á', 'é', 'í', 'ú' in an effective way.

Reproduction URL

https://storz.pages.dev/app/myFiles/desc

Reproduction steps

1. Go to 'https://storz.pages.dev' (width < 975px)
2. Click on 'Sign in'
3. Enter a valid email
4. Once logged in, upload a file with special chars in the name
5. See that once uploaded, the conversion is not being managed in an effective way

Screenshots

No response

Logs

No response

Browsers

No response

OS

No response

Description

The project needs to be linted. It can use the Eslint linter and the AirBnB JavaScript standard guide.
To make sure that contributors use the same style and some best practices, lining the project is necessary.

Example

ESlint can enforce one to use const keyword when the variable is not meant to be reassigned and let, if otherwise.

Currently, all the active API routes are in a single folder. It would be better to organize those routes and maintain a proper folder structure.

Description

Pressing enter on sign-in/log-in doesn't do anything. After looking briefly on the sign-in/login-in modal, I think the problem is that it's not using a form. I think this is bad for overall user experience.

Reproduction URL

https://github.com/chaaals/Storz

Reproduction steps

1. Go to sign-in/log-in page
2. Simply type in your information
3. Upon clicking enter~
4. It doesn't do any action

Screenshots

No response

Logs

No response

Browsers

No response

OS

No response

File detail doesn't include the time the file was created or uploaded. It only contains the date

Description

The files tab has a very squished look on a phone. Some margins and separations might be added to keep it clean.
Here's what it currently looks like.

Reproduction URL

https://storz.pages.dev/app/myFiles

Reproduction steps

1. Go to 'https://storz.pages.dev/app/myFiles'
2. Switch to a smaller screen in devtools
4. See the issue

Screenshots

![DESCRIPTION](LINK.png)

Logs

No response

Browsers

Firefox, Chrome, Safari, Microsoft Edge

OS

Windows, Linux

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

Adding a login button instead of use now because it is more intuitive and the website becomes easy to use. Please assign me this issue. #hacktoberfest

Basic Example

Drawbacks

We have to remove the use Now button

Unresolved questions

No response

Current code base is using local storage for storing the DID token, however it is not a good practice as it can be accessed by browser's JavaScript. Instead of that we can use httpOnly cookies to store them.

Description

Navbar is wrapped inside a div whereas it should be wrapped inside nav semantic HTML tag in Navbar.jsx

Reproduction URL

https://github.com/its-puneet/Storz/blob/main/client/src/components/navbar/Navbar.jsx

Reproduction steps

1. Go to 'main repo'
2. Click on 'Fork'
3. Go to "Reproduction URL"
4. See error

Screenshots

![DESCRIPTION](LINK.png)

Logs

No response

Browsers

No response

OS

No response

Given that the routes have grown and will be growing in the future. It is necessary to have

1. a single file where all the routes are used.
2. a single middleware file where all the middleware and the route middleware are used/registered

This way route and middleware are separated from the index file, thereby making it smaller, and cleaner.

Example:

-middleware/
--error.middleware.js
--another.middleware.js
--index.middleware.js // import all middleware and use them and export an instance of all the used middleware

Same for routes.

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

Can we add about page for this I need to discuss content for about page

Basic Example

we can add about app what does app do team photos in about

Drawbacks

Need for content help

Unresolved questions

No response

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

Create new files where we insert business logic from routes. I mean, pull that blocks of code where it's used mongoose's models out and put it in services files. If you agree with this request, I'd appreciate you can assign me this task.

Basic Example

A common example is when a User is created, that logic which we used, encapsulates in a function from service, then call it in the routes/services. In that way, we are just concerned about calling the function and not how it works.

Example:

Before

const user = new User({
            magic_id: magic_id,
            email: email,
            user_name: user_name,
            encryption_key: encryption_key,
            files: []
        })
        await user.save();

After

const user = UserService.create(magic_id, email, user_name, encryption_key, files);

Drawbacks

Decoupling services are essential for the app, in that way we can become the code more scalable and easier for testing, if the controller handled the whole application's logic and the request-handling logic, it would start to become really hard to test. Furthermore if at some point we want to migrate our database just we would have to create a new file and write the logic with the new database.

Unresolved questions

No response

Start Date

No response

Implementation PR

🐛 There is no button to go back from signin page , assign this task to me

Reference Issues

No response

Summary

add go back option

Basic Example

it would be better to have go back option

Drawbacks

user will not be able to go back one's he/she go to that signin page, yes there is option to go back using system functions , but it would be better to have go back option

Unresolved questions

No response

Add a dynamic 3D globe in the hero section of the landing page instead of the static globe png. (try to use three.js)

Reference repo link: https://github.com/nenadV91/Threejs-globe

Color: #00ffa8

Reference image:

A function that should be able to create to catch errors and show appropriate messages.

Start Date

09/04/2022

Implementation PR

No response

Reference Issues

No response

Summary

• Want to add some middleware that will prevent cross-site scripting and NoSQL query injection.

• Also want to implement a global error handler, which means after catching errors all over the application we send the error to a global error handler which will decide what to do with it.

Basic Example

Let on user.js we get an error which will go to catch section

e,g

try{
   //......
  }catch( err) {
      next( err) // calling the global err handler
  }

And in index.js we have the global error handler which will be called

index.js

app.use( (err, req, res, next) => {
          res.status( err.statuscode) . json( {
          data: err.message
          })
});

Drawbacks

There is no drawback of adding above features

Unresolved questions

No response

Description

Trying to Login and getting this error in console.

Reproduction URL

https://github.com/anomic30/Storz/

Reproduction steps

1. Go to 'LogIn'
2. Click on 'Login button'
3. Open console
4. See error

Screenshots

Logs

No response

Browsers

Chrome

OS

Windows

Description

Having the navBar sticky, especially on the login and after the login page (upload part) will help users navigate easier and look cleaner.

Reproduction steps

1. Go to 'https://storz.pages.dev/'
2. Scroll down
4. See the navBar is gone and there is no option to sign in or upload.

Screenshots

No response

Logs

No response

Browsers

Chrome

OS

Windows

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

The current structure of the API routes has the controller functions and its data accesing tightly coupled together. This development design will not allow further easy integration to any other database that is not mongodb/mongoose without changing the entire API route functions. Suggestion:
[] A separate folder for all the controllers
[] A separate folder as the data repository (where different databases data query/ mutaions functions are define)

Basic Example

For example
The route that login in the user first queries the data dabase using the mongoose findOne method on the User Object... this direct calling of mongoose method within the route controllers had directly coupled the sytem to mongodb/mongoose as the defualt database. The challenge to this is that in the nearest future if other other databases are to be integrated into the system, the change will not only be done on the database connecttion config file but also across the entire routes controllers thereby making such integration a diffult one. But having a separate repository that is resposing for the direct communication of the database method before being used by the controller will enable easy inegration.

Drawbacks

This development design will not allow further easy integration to any other database that is not mongodb/mongoose without changing the

Unresolved questions

No response

In this issue I just want to include a small detail in the readme of the project

• Rename the env.example file
• Rename the file in both client and server folder

I raised this issue as i faced an error which got resolved after renaming the files. So either we can rename the files or inform the contributor in readme about renaming the file.

Description

Currently the Files section actives the horizontal scrolling when a file name is too big

Reproduction URL

https://storz.pages.dev/app/myFiles

Reproduction steps

1. Go to the Storz app https://storz.pages.dev/
2. Login
3. Add a new file with a large name
4. See behavior

Screenshots

![image](https://user-images.githubusercontent.com/48018975/193726016-74c69263-7c1f-4f13-8045-83a1b36d005f.png)

Logs

No response

Browsers

Chrome

OS

Windows

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

I want to create a scroll back to top button for this website, as it is of multiple pages.
I want to make this contribution under hacktoberfest'22.

Basic Example

My feature will help in returning back to the landing page, once the button gets clicked.

Drawbacks

My feature will help in returning back to the landing page, once the button gets clicked.

Unresolved questions

No response

Add mobile as well as tablet view responsiveness to the main app.
Make sure to follow the proper CSS naming conventions.

Description

If the user is hovering over Storz image in the nav menu and clicks it, it doesn't do anything. Remove the hover or add a scroll up event

Reproduction URL

https://storz.pages.dev/

Reproduction steps

1. Go to https://storz.pages.dev/
2. Hover the Storz image in the nav menu on the top left
3. Click it
4. Nothing happens

Screenshots

No response

Logs

No response

Browsers

Chrome

OS

Windows

Start Date

02-10-2022

Implementation PR

No response

Reference Issues

Every time i logout for logging in again i have to click on the magic link there might a a need to have a simple user password system along with the magic link which might be useful for regular users to not have to go and click on the magic link everytime they need to login

Summary

implementing a email password verification system along with the magic link system

Basic Example

a seperate page and seperate routes and controllers to for loggin in with email and password instead of magic link

Drawbacks

a lot of small changes might have to be made throughout the project as a new authentication system is being added

Unresolved questions

No response

Description

Each and every time I am uploading a file it is showing me the below image. what that x sign mean and there were no error message as well.

Reproduction URL

no

Reproduction steps

1. Go to the website
2. Click on upload files
4. See error

Screenshots

Logs

No response

Browsers

Chrome

OS

Linux

Description

The files information page has a bad responsive design and on smaller screens/devices, the website will start to break. I believe that the root cause of this is that at smaller screens, the buttons will run out of room and start invading the padding and margin of its parent div. Allowing the button to wrap at a smaller size, it can provide it with more room and space to work with on a new line.

Here is what it looks like currently

At 420px width

At 320px width

This is my proposed design:

Reproduction URL

https://storz.pages.dev/app/myFiles/desc

Reproduction steps

1. Go to https://storz.pages.dev/app/home
2. Upload a file to the IPFS network
3. Go to https://storz.pages.dev/app/myFiles and open the file
4. Open devtool and adjust width to less than 420px
4. See the issue

Screenshots

See in description

Logs

No response

Browsers

No response

OS

No response

Description

Whenever the app reloads it goes to the login page and stays for 2-3 seconds before redirecting to the target page.

Reproduction URL

https://storz.pages.dev/

Reproduction steps

1. Login 
2. Go to my files and reload the page.

Screenshots

No response

Logs

No response

Browsers

No response

OS

No response

Start Date

09/04/2022

Implementation PR

No response

Reference Issues

No response

Summary

Below are some of my proposed changes

Controller

1. Create a Controller folder, which contains controller files
2. There is a controller file for each route
   e,g -
   route/user.js <-- controller/userController.js
   route/upload.js <-- controller/uploadController.js
3. Each controller file contains the route handler function specific to its corresponding route endpoint

Misc

1. Merge the auth.js route to the user.js route, keeping all the authMiddleware still working. Reason for that these two route share the same API endpoint, /api/user/
   e,g - auth --> /api/user/login
   user --> /api/user/files

Basic Example

File Structure after refactoring into MVC would look like this -

controller/
    userController.js
    uploadController.js
    downloadController.js
    
model/
   
routes/
    user.js
    download.js
    upload.js
    test.js

index.js
package.json

Drawbacks

Doing Refactoring to MVC only increases the require() lines and increases the number of files. Except that there is no other problem

Unresolved questions

No response

Start Date

No response

Implementation PR

No response

Reference Issues

No response

Summary

A rate limiter can be added to the backend to avoid slowloris attacks.

Basic Example

This would avoid unnecessary open requests to server

Drawbacks

It would limit requests per second from same ip (value can be changed though)

Unresolved questions

No response

What if we add Docker to this project so that anyone can spin up the entire project without any hustle. Please let me know your thoughts on this.

There is a small overflow issue. You can scroll horizontally in both the web and mobile versions.

Currently, we are using a custom npm package for the encryption/decryption of the files. This package doesn't have proper asynchronous methods. So we would like to create our own custom functions which are asynchronous and error-free. These functions should be able to catch errors and show the appropriate message properly.

• Want to add a popup modal which appears when the user clicks on the Logout button.

ISSUE

• Currently User logsout Immeadiately the button is clicked.
• If done by mistake . This leads the user to login again via confirmation through his gmail account.

Improvements

• When user clicks the logout button . A popup must appear asking the user if he wants to logout?
• This way prevents the user to logout by mistake.
• Improves user experience .

Add a sorting feature to the My Files page.

For example:

• Sort by Upload date ( ascending or descending )
• Sort by File names ( alphabetically )
• Sort by File size ( smaller or larger )

The error handler and how errors are handled need to be fixed.

• The global error handler has an issue. To reproduce the error, once the server is running visit a route that does not exist, and the request will hang on until the server timeout. Example: visit: http://localhost:8080/api/user or http://localhost:8080/this-does-not-exist
• Asynchronous errors need to be handled: currently on the route try ... catch were used to hand asynchronous errors. We can get rid of it and make the code more cleaner and maintainable. This way once can 'throw new Error' and it would be handled, also an unhandled exception would be captured. This will help us remove multiple try...catch in all the route handlers.

I want to add valuable contributors to readme.
Please assign it to me under hacktoberfest.

The project needs a pre-hook commit that will run to lint and format the change to the project before it gets committed to git.

Given that linting and formatting have been configured for the project, setting up a pre-commit hook will help lint and format changes on committing.