colouring-cities / colouring-britain

Developed out of the Colouring London prototype. Collecting data on Britain's buildings and testing new core features
https://colouringbritain.org/
GNU General Public License v3.0
10 stars 2 forks source link

Ampersand (&) in text fields cannot be copied #121

Open polly64 opened 2 years ago

polly64 commented 2 years ago

Trying to copy all selected on Worship street (shown coloured under Team section. says invalid parameter- seems quite random - looking at buildings around liverpool street station sometimes works sometimes doesn't

edwardchalstrey1 commented 2 years ago

@polly64 would you be able to screenshot this as I'm not sure what you mean? You can drag images including saved screenshots into the comment boxes on GitHub

polly64 commented 2 years ago

invalid parameter screenshot

edwardchalstrey1 commented 2 years ago

Ok thanks Polly, I haven't managed to replicate the problem yet - do you remember one of the building IDs where this happened or can you screenshot the values that are being copied?

polly64 commented 2 years ago

ID worship street

polly64 commented 2 years ago

@edwardchalstrey1 in meetings til 12 but do send me any other queries

edwardchalstrey1 commented 2 years ago

Ok, on the 2nd one, it's not liking that you used an ampersand & <- I'll take a look why this is, doesn't seem like we should reject all alphanumeric characters

polly64 commented 2 years ago

@edwardchalstrey1 ah ok thanks, also I am free til 11 am now

polly64 commented 10 months ago

@mdsimpson42 can you check if this fixed

mdsimpson42 commented 10 months ago

It should be less of an issue now we don't allow free text fields. It seems to work for some buildings, but if I try to copy the data for Euston (which has "Richard Seifert & Partners" as the Main Building -> Designer), then I do get the "Invalid Parameters" error.

polly64 commented 10 months ago

@mdsimpson42 let's chat with MK re this can you put on your list for our next meeting with him thanks