search

edgedb / easy-edgedb

An illustrated textbook designed to be a one-stop shop for learning EdgeDB
https://www.edgedb.com/easy-edgedb
76 stars 39 forks source link

Can not create the property 'title' on NPC before it exists. #113

Closed bryanjnelson closed 1 year ago

bryanjnelson commented 1 year ago

In the section Using the ‘for’ and ‘union’ keywords in Chapter 9 the following statement is mentioned:

update NPC filter .name = 'John Seward' set { title := 'Dr.' };

Perhaps NPC.title existed at some point in another iteration of this documentation, but at the current time it does not exist by Chapter 9.

Potential solutions:

  1. Remove this statement
  2. Add NPC.title to the schema in a previous chapter
  3. Move this statement to a later point in the book where NPC.title does exist (Chapter 10 I believe)
Dhghomon commented 1 year ago

Yes, I noticed this last week during my ongoing review on this branch where it is fixed: https://github.com/edgedb/easy-edgedb/tree/self-review Hopefully will reach the end of it in a few days and able to merge it.

Dhghomon commented 1 year ago

The branch has been merged so it's been solved with solution #3 you mentioned, thanks!