Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Terminology Updates - Delete site dialog #2472

Open
1 task done
Tracked by #954
DerekCaelin opened this issue Nov 12, 2024 · 4 comments
Open
1 task done
Tracked by #954

Terminology Updates - Delete site dialog #2472

DerekCaelin opened this issue Nov 12, 2024 · 4 comments
Assignees
@DerekCaelin
Labels
Engineering
Milestone
LandPKS Backlog

Comments

@DerekCaelin
Copy link
Contributor

DerekCaelin commented Nov 12, 2024 •
edited by paulschreiber
Loading

Update Terminology in the Delete Sites dialog window.

Design

image
Figma

Acceptance Criteria

Delete Site dialog

  • of its data -> its observations
@DerekCaelin DerekCaelin mentioned this issue Nov 12, 2024
Open
@DerekCaelin DerekCaelin transferred this issue from techmatters/terraso-product Nov 12, 2024
@github-project-automation github-project-automation bot moved this to Todo in LandPKS Nov 12, 2024
@DerekCaelin DerekCaelin added this to the LandPKS Backlog milestone Nov 12, 2024
@paulschreiber paulschreiber moved this from Todo to In Review in LandPKS Nov 18, 2024
paulschreiber added a commit that referenced this issue Nov 18, 2024
@paulschreiber
fix: update delete site dialog text (#2472)
6b565bd
@paulschreiber paulschreiber mentioned this issue Nov 18, 2024
Merged
paulschreiber added a commit that referenced this issue Nov 18, 2024
@paulschreiber
fix: change data to observations throughout app (#2495)
e444fbc 
* fix: update delete account screen #2474

* fix: update interstitial text (#2473)

* fix: remove duplicated delete site text

* fix: update delete site dialog text (#2472)

* fix: fix transfer sites screen text (#2469)

* fix: use cannot instead of can’t for internal consistency

* fix: update delete project text (#1150)

* fix: update team management text (#2468)

* fix: update depth text (#2467)

* fix: update projects screen text (#2466)

* fix: update data privacy text (#2465)

* fix: update requirements text (#2464)

* chore: update snapshots

* chore: generate PO files
@paulschreiber paulschreiber moved this from In Review to QA in LandPKS Nov 18, 2024
@DerekCaelin
Copy link
Contributor Author

Delete sites dialog modal now looks like this:
Image

@DerekCaelin DerekCaelin moved this from QA to Todo in LandPKS Nov 19, 2024
paulschreiber added a commit that referenced this issue Nov 20, 2024
@paulschreiber
fix: correct key for site deletion strings #2472
824ec92
@paulschreiber paulschreiber mentioned this issue Nov 20, 2024
Merged
@paulschreiber paulschreiber moved this from Todo to QA in LandPKS Nov 20, 2024
@paulschreiber
Copy link
Member

Please re-verify in 210 or later.

paulschreiber added a commit that referenced this issue Nov 20, 2024
@paulschreiber
fix: fix en.json string typo and incorrect key in tsx file (#2509)
bfe3a1b 
* fix: correct typo in 'those' #2469

* fix: correct key for site deletion strings #2472

* fix: update text for change depths dialog

* fix: correct typo in ‘their’
@DerekCaelin
Copy link
Contributor Author

The delete project site dialog is correct, but the delete unaffiliated site dialog now makes references to projects:

@paulschreiber paulschreiber moved this from QA to In Review in LandPKS Nov 26, 2024
@paulschreiber paulschreiber mentioned this issue Nov 26, 2024
Merged
@paulschreiber paulschreiber moved this from In Review to QA in LandPKS Nov 27, 2024
@paulschreiber
Copy link
Member

Please verify in 214 or later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DerekCaelin DerekCaelin

Labels
Engineering
Projects
LandPKS
Status: QA
Milestone
LandPKS Backlog
Development

No branches or pull requests
2 participants
@paulschreiber @DerekCaelin