Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Character Application Manager improvements #4

Open
4 of 18 tasks
LikeLakers2 opened this issue Jan 25, 2017 · 1 comment
Open
4 of 18 tasks

Character Application Manager improvements #4

LikeLakers2 opened this issue Jan 25, 2017 · 1 comment

Comments

@LikeLakers2
Copy link
Owner

LikeLakers2 commented Jan 25, 2017

I will attempt to include these in the rewrite.

Suggestions

  • rp!giveme should give roleplayer if they already have a accepted character (This is in RoleGiver but it applies anyways)
  • rp!view accept[ed]/rp!list accept[ed] and other equivalents to view characters with those statuses
  • (Suggested by Asriel Dreemurr#8310) Registration process should include a message referencing the ability to load existing characters into registration, such that people may re-use old IDs.
  • Implement timestamps and perhaps a log of actions done to change characters.
  • Name searching (for when someone specifies a character name) should first do exact matching, then contains matching.
  • Name searching should search only through one's own characters when doing rp!set, unless the user is a admin.

Please suggest more if you have any suggestions.

TODO

  • Deprecate rp!cm and opt for using the dedicated commands -- leave rp!cm in for some time for backwards compatibility but warn about the command being deprecated.
  • Consolidate various methods into Character, CharacterDB, Page and Wiki classes. (Ideally, we want to implement them in Page and Wiki, and use overwriting methods in Character and CharacterDB if we need to change them.)
  • Rewrite rp!charmanage functions to use these various methods.
    • rp!view
    • rp!set
    • rp!setprop
    • rp!list
    • rp!search
    • rp!delete
    • Test all the new commands to make sure they work
  • Rewrite CharAppManager to use load to the new Character and CharacterDB classes which will be customized specifically to the needs of CAM.

(To be expanded)

When we're done

  • Set a day where CAM will not be in use. This day will be used to transfer all the original character files over a new format:
{
  "charid": 4,
  "properties": {
    "ownerid": 98296942768967680,
    "Status": "Pending"
  },
  "fields": {
    "Name": "1",
    "Age": "Undefined",
    "Gender": "None",
    "Basic Appearance": "Insert dummy text here.\nSecond dummy text line.",
    "Brief Bio": "Just your typical test user.",
    "Xyzzy": "Plugh",
    "help": "You should probably put something here."
  }
}

(basically we try to move charid out of the properties block so i dont have to block it in setprop)

  • Make sure it all works and prepare the floodgates.
@LikeLakers2
Copy link
Owner Author

Created a PR for this at #7 -- ignore the milestone changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant