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.

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

Style/remove lorem ipsum #198

Merged
merged 2 commits into from
Oct 19, 2024
Merged

Style/remove lorem ipsum #198

merged 2 commits into from
Oct 19, 2024

Conversation

xk0niR
Copy link
Contributor

@xk0niR xk0niR commented Oct 18, 2024

close #169

I've also fixed some other minor mistakes in some text.

Also switched position of some text inside TreeGeneralData.tsx so Latitude and Longitude are next to each other (see screenshot).

grafik

@xk0niR xk0niR requested a review from doriengr October 18, 2024 21:40
@doriengr
Copy link
Contributor

@xk0niR I think this should be merged in release01 and not in develop.

@doriengr doriengr changed the base branch from develop to release/v1.0.0 October 19, 2024 07:05
frontend/src/components/tree/TreeUpdate.tsx Outdated Show resolved Hide resolved
frontend/src/routes/_protected/profile.tsx Outdated Show resolved Hide resolved
frontend/src/routes/_protected/settings/import.tsx Outdated Show resolved Hide resolved
frontend/src/routes/index.tsx Outdated Show resolved Hide resolved
@doriengr
Copy link
Contributor

Some descriptions need some editing:

  • Description of index page of all tree clusters
  • Description of vehicles
  • Description for team members

@xk0niR xk0niR added this to the Release 1 milestone Oct 19, 2024
@xk0niR
Copy link
Contributor Author

xk0niR commented Oct 19, 2024

Please review again. I've changed every comment and also added descriptions for index page of all tree clusters, Description of vehicles, Description for team members

Some info for the change inside index page of all tree cluster:
I've removed numbers (40 trees, atleast 5 trees with sensors) to be more "open" (e.g. someone wants to create a tree cluster with just 3 trees and all have sensors)

@xk0niR xk0niR requested a review from doriengr October 19, 2024 10:46
@xk0niR xk0niR merged commit b197899 into release/v1.0.0 Oct 19, 2024
@xk0niR xk0niR deleted the style/remove-lorem-ipsum branch October 19, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Replace Lorum Ipsum texts with correct content
2 participants