Accepting Suggestions about discord-xp HOT 24 OPEN

Can I help rewrite a new version of discord-xp, wherein you could do:

(require('discord-xp')).setDB('quick.db', quickdb)
// quickdb is the quick.db database.
// basically what I mean is support multiple databases

Instead of the static async setURL function, you could use this:

  static async setDB(dbname, db) {
  const databases = ['quick.db', 'mongoose']
  if (!databases.includes(dbname)) throw new TypeError(`Provided database (${dbname}) is not valid.`)
  if (database.includes('mongoose')) {
      mongoUrl = dbUrl;
    return mongoose.connect(dbUrl, {
      useNewUrlParser: true,
      useUnifiedTopology: true,
      useFindAndModify: false
    });
    }
  }

from discord-xp.

Types for typescript

from discord-xp.

Types for typescript

That was implemented today, several minutes ago with the new version v1.1.11. Check the instructions in the README.md on how to update (https://npmjs.org/discord-xp).

from discord-xp.

Types for typescript

That was implemented today, several minutes ago with the new version v1.1.11. Check the instructions in the README.md on how to update (https://npmjs.org/discord-xp).

O minutes ago, okay thanks.

from discord-xp.

I think it would be nice if there's event emitters, for example the library would emit an event when someone leveled up or down

from discord-xp.

Maybe can let it using mysql or mariadb
Not everyone use mongo

from discord-xp.

Can I help rewrite a new version of discord-xp, wherein you could do:

(require('discord-xp')).setDB('quick.db', quickdb)
// quickdb is the quick.db database.
// basically what I mean is support multiple databases

Instead of the static async setURL function, you could use this:

  static async setDB(dbname, db) {
  const databases = ['quick.db', 'mongoose']
  if (!databases.includes(dbname)) throw new TypeError(`Provided database (${dbname}) is not valid.`)
  if (database.includes('mongoose')) {
      mongoUrl = dbUrl;
    return mongoose.connect(dbUrl, {
      useNewUrlParser: true,
      useUnifiedTopology: true,
      useFindAndModify: false
    });
    }
  }

I was going to suggest the same. Add quick-db / simple json or sqlite support, via other databases

from discord-xp.

Can I help rewrite a new version of discord-xp, wherein you could do:

(require('discord-xp')).setDB('quick.db', quickdb)
// quickdb is the quick.db database.
// basically what I mean is support multiple databases

Instead of the static async setURL function, you could use this:

  static async setDB(dbname, db) {
  const databases = ['quick.db', 'mongoose']
  if (!databases.includes(dbname)) throw new TypeError(`Provided database (${dbname}) is not valid.`)
  if (database.includes('mongoose')) {
      mongoUrl = dbUrl;
    return mongoose.connect(dbUrl, {
      useNewUrlParser: true,
      useUnifiedTopology: true,
      useFindAndModify: false
    });
    }
  }

I was going to suggest the same. Add quick-db / simple json or sqlite support, via other databases

MrAugu will add support for different storage providers including json if I recall that correctly.

from discord-xp.

@Dragonizedpizza Go ahead and create a pull request.

from discord-xp.

Can I help rewrite a new version of discord-xp, wherein you could do:

(require('discord-xp')).setDB('quick.db', quickdb)
// quickdb is the quick.db database.
// basically what I mean is support multiple databases

Instead of the static async setURL function, you could use this:

  static async setDB(dbname, db) {
  const databases = ['quick.db', 'mongoose']
  if (!databases.includes(dbname)) throw new TypeError(`Provided database (${dbname}) is not valid.`)
  if (database.includes('mongoose')) {
      mongoUrl = dbUrl;
    return mongoose.connect(dbUrl, {
      useNewUrlParser: true,
      useUnifiedTopology: true,
      useFindAndModify: false
    });
    }
  }

I was going to suggest the same. Add quick-db / simple json or sqlite support, via other databases

MrAugu will add support for different storage providers including json if I recall that correctly.

Alright cool

from discord-xp.

I think you should make an function like
giving the rank of the user instead of fetching all the leaderboard you could simply do this

user.rank()
// gives the rank of the user

from discord-xp.

I think you should make an function like
giving the rank of the user instead of fetching all the leaderboard you could simply do this

user.rank()
// gives the rank of the user

First of all you'll always need to sort through the entries to get the rank/ position.
Second the fetch() has been updated and now takes a third boolean parameter called fetchPosition. If you enable that then

console.log(<user>.position);   // Position of the user

from discord-xp.

@Dragonizedpizza Go ahead and create a pull request.

Gonna work on it rn

from discord-xp.

@Myst82015
@AdamTmHun
#35

from discord-xp.

I think you should make an function like
giving the rank of the user instead of fetching all the leaderboard you could simply do this

user.rank()
// gives the rank of the user

First of all you'll always need to sort through the entries to get the rank/ position.
Second the fetch() has been updated and now takes a third boolean parameter called fetchPosition. If you enable that then

console.log(<user>.position);   // Position of the user

The fetchPosition is the same as

user.rank()

so my wish has been granted

from discord-xp.

allow us to add roles (eg. user reaches level 5 and gets the role "level 5") and make it so that
users can change the background of their rank card :)

from discord-xp.

both of those can be done yourself though

from discord-xp.

how?

from discord-xp.

how?

just check if the level the user got to is level 5, and discord-xp doesn't provide the rank card itself

from discord-xp.

Maybe update the examples so the code can run in discord.js v13 as well

from discord-xp.

Maybe update the examples so the code can run in discord.js v13 as well

I'll take a look as soon as I have some free time.

from discord-xp.

Maybe update the examples so the code can run in discord.js v13 as well

I'll take a look as soon as I have some free time.

Alr tysm, take a look at that .setStatus(..) part for canvas as well, if member is offline, then canvas says .setStatus cannot be null

from discord-xp.

how can i check if a user is level 5 or 6 and give them a role from the bot?

from discord-xp.

how can i check if a user is level 5 or 6 and give them a role from the bot?

this is meant for suggestions, not support

from discord-xp.