TBD54566975 / dwn-server

DWN server - host anywhere publicly accessible for DIDs to use
Apache License 2.0
40 stars 41 forks source link

Document methods in the repo #64

Closed adam4leos closed 12 months ago

adam4leos commented 1 year ago

Background:

We're (unfortunately) lacking method docs, and having some would be amazing.

Task Details:

Consider covering methods in the repo, especially public and less straightforward, with some jsdocs style comment

example

/**
   * Handler for opening websocket event - `connection`.
   * Sets listeners for `message`, `pong`, `close`, and `error` events.
   */
#handleConnection(socket: WebSocket, _request: IncomingMessage): void {
  const dwn = this.dwn;
  // ...rest of the code...

Picking Up This Issue:

Questions:

Resources:

Remember, communication is key! If you have any questions or face any challenges, we're here to help so please don't hesitate to reach out.

Good Luck! 🍁

MaticSulc commented 1 year ago

Picking this up! :)

EbonyLouis commented 1 year ago

@MaticSulc yay! I've assigned it to you!

I'd also like to note that Hacktoberfest officially kicks off on October 1st. Contributions made before the event may not be eligible for the challenge. However, feel free to start working on it, but I'd recommend holding off on submitting your PR until the event begins.

to be eligible for Hacktoberfest's official prizes make sure you're registered, registration is now OPEN.

EbonyLouis commented 1 year ago

Hey @MaticSulc would you still like to work on this issue?

KhoiUna commented 1 year ago

Hey @MaticSulc would you still like to work on this issue?

Can I pick this one up if it's not being worked on? @EbonyLouis

EbonyLouis commented 1 year ago

Hey @KhoiUna yes you can work on it, thanks!

KhoiUna commented 12 months ago

@EbonyLouis Submitted a PR