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

Removing texts part c#161 #284

Merged
merged 5 commits into from
Dec 9, 2023
Merged

Removing texts part c#161 #284

merged 5 commits into from
Dec 9, 2023

Conversation

amabelleS
Copy link
Collaborator

Description

Removed from: FilterPositionsByStartTimeSelector, GapsPage, DisplayGapsPercentage, DateSelector, and TimelinePage.

@amabelleS amabelleS requested a review from NoamGaash as a code owner December 8, 2023 21:36
@@ -94,5 +94,7 @@
"bug_type_message":"Please enter the type of request",
"bug_type_bug":"Bug",
"bug_type_feature":"Feature Request",
"funding_paragraph": "The Databus project is developed by the Hasadna for public knowledge, by the contributions of volunteers, and based upon"
"funding_paragraph": "The Databus project is developed by the Hasadna for public knowledge, by the contributions of volunteers, and based upon",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space removed by purpose?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space removed by purpose?

No. Maybe Lint complained, but I can't even see what space we are talking about... just that there is a change in "funding_paragraph" that I did not mean to make.

@@ -64,6 +39,32 @@ const GapsPage = () => {
const [gapsIsLoading, setGapsIsLoading] = useState(false)
const [onlyGapped, setOnlyGapped] = useSessionStorage('onlyGapped', false)

function formatTime(time: Moment) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you move it inside the component?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you move it inside the component?

I feel ashamed about why, and I am trying to think of another solution.
You can call t / const { t } = useTranslation() outside of a react component because it is a hook. So I moved the functions that use t inside of the react component function. Very dirty, I need a shower now:)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you're right. Its not dirty - I just didn't expect moving the time format to the translation file

@shootermv shootermv merged commit c293bd6 into main Dec 9, 2023
9 checks passed
@shootermv shootermv deleted the removing-texts_part-c#161 branch December 9, 2023 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants