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

correcting wrong time showing in grid view #34179

Merged

Conversation

Abhishek-kumar-samsung
Copy link
Contributor

@Abhishek-kumar-samsung Abhishek-kumar-samsung commented Sep 7, 2023

related: #32751

corrected inconsistent times in grid view.
I have kept the code of time on area above bars same as the time of tooltip, so that there would be no inconsistency.

Earlier photos( It can be seen in photo that tooltip date is 6 Sep, but date above bar is 5 Sep ):
normal_hover_time

normal_wrong_time

Photo after correction
corrected_time

@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Sep 7, 2023
@Abhishek-kumar-samsung
Copy link
Contributor Author

@potiuk @Taragolis @eladkal @pierrejeambrun @bbovenzi @ryanahamilton @ashb

There was some inconsistency in timings shown in grid view as per related issue.
I have corrected that by using similar solution discussed in same issue.
Can you pls check and review.

@bbovenzi
Copy link
Contributor

bbovenzi commented Sep 8, 2023

Looks good. Just run yarn lint to fix some formatting

@Abhishek-kumar-samsung
Copy link
Contributor Author

@bbovenzi lints passed,

Can you pls check

Looks good. Just run yarn lint to fix some formatting

@eladkal eladkal added the type:bug-fix Changelog: Bug Fixes label Sep 8, 2023
@eladkal eladkal added this to the Airflow 2.7.2 milestone Sep 8, 2023
@bbovenzi bbovenzi merged commit 8871f95 into apache:main Sep 12, 2023
ephraimbuddy pushed a commit that referenced this pull request Oct 5, 2023
* correcting wrong time showing in grid view

* lint errors resolved

* lint errors resolved

(cherry picked from commit 8871f95)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants