Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an uniq id for characters + full character object in dialogue #3

Open
jrmgx opened this issue Jul 27, 2016 · 9 comments
Open

Add an uniq id for characters + full character object in dialogue #3

jrmgx opened this issue Jul 27, 2016 · 9 comments

Comments

@jrmgx
Copy link
Contributor

jrmgx commented Jul 27, 2016

On the client side we need to rely on a unique character id for some feature
this change will affect both plays and play/[*]/act/[*] endpoints in multiple places

It will allow us to have a better support on the LBG play

The id must be unique on a specific play, there is no other constrain IMHO

Plan

  • Add a unique character id in the plays endpoint
[{
  "id": "fourberies",
  "title": "Les Fourberies de Scapin",
  "illustrationFile": "/play/fourberies/illustration.png",
  "characters": [{
    "identifier": "argante",
    "name": "ARGANTE",
    "desc": "père d’Octave et de Zerbinette.",
    "avatarFile": "/play/fourberies/avatars/argante.png"
  }
  • Add a unique character id in the play/*/act/* endpoint on the characters listing part
{
  "title": "Le Bourgeois gentilhomme",
  "characters": [{
    "identifier": "monsieur-jourdain",
    "name": "MONSIEUR JOURDAIN",
    "desc": "bourgeois."
  }]
  • Add a full character reference in the play/*/act/* endpoint on the dialogue part
    This include a clear separation between the character and the information on his sentence
    (in this example "MAÎTRE DE MUSIQUE parlant à ses Musiciens") has been moved to an action field
"dialogue": [{
  "character": {
    "identifier": "maintre-de-musique",
    "name": "MAÎTRE DE MUSIQUE"
  },
  "action": "MAÎTRE DE MUSIQUE parlant à ses Musiciens",
  "text": "Venez, entrez dans cette salle, et vous reposez là, en attendant qu’il vienne.",
  "textWithEmoji": "Venez, entrez dans cette salle, et vous reposez là, en attendant qu’il vienne.",
  "emojiWords": { }
}

If this issue is not clear let's talk more about it

@adrienjoly
Copy link
Owner

adrienjoly commented Jul 27, 2016

ahah! t'es au taquet, jérôme! ^^

@jrmgx
Copy link
Contributor Author

jrmgx commented Jul 27, 2016

Il faut que je note les trucs sinon ça m'occupe la tête pour rien :)

@adrienjoly
Copy link
Owner

adrienjoly commented Jul 27, 2016

GTD style! tu as raison!

@adrienjoly
Copy link
Owner

Why do you need unique ids for characters?

@jrmgx
Copy link
Contributor Author

jrmgx commented Aug 3, 2016

it is mainly to group bubbles by characters
it is also used to know if it is the character you selected that is talking or not

@adrienjoly
Copy link
Owner

Well, names are unique too in the scope of a play, no?

Le mer. 3 août 2016 18:43, Jérôme Gangneux notifications@github.com a
écrit :

it is mainly to group bubbles by characters
it is also used to know if it is the character you selected that is
talking or not


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#3 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgdRb6GJ7wFxidB3HibTpbSCAk7_4Tdks5qcJjMgaJpZM4JV9YG
.

Adrien Joly
adrienjoly.com

@jrmgx
Copy link
Contributor Author

jrmgx commented Aug 4, 2016

nop because of other information that comes with the name
like "MONSIEUR JOURDAIN, l'épé à la main" or anything else.
If you can make sure that a name is really a name it will work (as you say)

@adrienjoly
Copy link
Owner

ok. will it work better for you if I store only the name in the character
field, and other indications (e.g. l'épée à la main) in an indications
field ?

On Thu, Aug 4, 2016 at 9:41 AM Jérôme Gangneux notifications@github.com
wrote:

nop because of other information that comes with the name
like "MONSIEUR JOURDAIN, l'épé à la main" or anything else.
If you can make sure that a name is really a name it will work (as you say)


You are receiving this because you commented.

Reply to this email directly, view it on GitHub
#3 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgdRQBm_mhfqPVSNxNwT0T-RxaU5Rg8ks5qcZe6gaJpZM4JV9YG
.

Adrien Joly
adrienjoly.com

@jrmgx
Copy link
Contributor Author

jrmgx commented Aug 4, 2016

will be great

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

No branches or pull requests

2 participants