techmatters / terraso-mobile-client

A React Native project for Terraso’s LandPKS mobile application.
https://landpks.terraso.org
GNU Affero General Public License v3.0
2 stars 0 forks source link

Site name in app bar updates before "Save" button is pressed #1658

Open knipec opened 3 months ago

knipec commented 3 months ago

The site name in the app bar updates as the user edits the name field, rather than after the save button is pressed.

Steps to reproduce

  1. Go to settings for a site
  2. Edit the name
  3. see issue

Expected behavior

The site name in the green app bar at the top doesn't change until after you press Save

Actual behavior

The site name appears to update in the app bar as you type. But it's not officially updating the real site name until you press Save.

Additional

paulschreiber commented 3 months ago

The current behaviour is fine.

DerekCaelin commented 2 months ago

Curious about @CourtneyLee333's opinion about this issue

CourtneyLee333 commented 2 months ago

I agree with Carissa's assessment. The name in the app bar should not update until Save.

knipec commented 2 months ago

@CourtneyLee333 @tm-ruxandra Do you consider this part of the components library work / part of #1784 ?

tm-ruxandra commented 2 months ago

@knipec I wouldn't, this seems like a bug with flow of data not the component itself.

DerekCaelin commented 2 months ago

Edited title and description, looking forward to triaging

knipec commented 1 month ago

Triage notes: