I had some problems because many changes are being made to the geometry nodes code.
I added the changes that @Hans Goudey (HooglyBoogly) asked for.
I took as reference the code used for node_shader_script, so that help me a lot.
I'm still getting familiar to all the parts involved with the nodes so I'm not sure if something is missing, so let me know.

• Fix to errors produced when merging the latest master

• Added layout callbacks to node_fn_input_string.cc to match with cfa48c84d06c

• Custom storage NodeInputString now uses char *
• Merge remote-tracking branch 'origin/master' into arcpatch-D10316

Updating D10316: Added New String Input Node

Here it is!
Let me know of anything that needs a fix or improvement.

Ok then.
I'll double check everything and I'll consult the developer intro on Best Practices and all that.
Thanks!

Hi guys!! I have it working now.

typedef struct NodeInputString {
  char string[1024];
} NodeInputString;

I saw the bNodeSocketValueString use char value[1024] but I think is Deprecated.

I'm on it. Thanks!

Great!! I already did some digging and I was wondering if the string node should be a Geometry node or instead a Function node (like the vector node) or a Shading node (like the value node).

Hi! Can I work on this?