Fixing issue with grid not allowing copying external ID

[mike12345567]

Description

Fixing an issue with ID/Rev not being copyable from context menu due to the way IDs were split/handled in the grid.

External rows can have row IDs which contain - characters, this is used in the grid store to attach the field which is currently focused, this can be avoided by assuming the last split off - will leave the ID as the first element and the field as the second element.

Addresses

• (Altering a external data source's primary key causes the _id and _rev number to become disabled - allow budibase to notify the user when their DB schema has deviated from the one currently stored #13586)

[mike12345567]

@aptkingston would appreciate your input on this - after some discussion with @samwho realised that we just can't use - for joining the ID and field name, as it could be present in either of them, so splitting is very difficult.

Instead using something very unusual to avoid this problem - ‽‽

[aptkingston]

LGTM!

Technically these are cell IDs rather than row IDs so a rename would be appreciated not definitely not required.

So purely for terminology consistency (since this is only for the grid codebase and we may as well use the same terms) I would opt for

const { rowId, columnName } = parseCellID(cellId)
const cellId = getCellID(rowId, columnName)

But that's just me being overly anal about naming. Functionally this looks perfect and is much more robust and maintainable. Meant to do this myself at some stage rather than have the duplicated logic all over the place. Nice one!