Skip to content

[RELEASE] Bump to v5.6.1 #1115

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

Closed
wants to merge 3 commits into from
Closed

[RELEASE] Bump to v5.6.1 #1115

wants to merge 3 commits into from

Conversation

dzole0311
Copy link
Collaborator

🎉 Features

🚀 Improvements

🐛 Fixes

**Related Ticket:** #1113

### Description of Changes
The change fixes the label formatting of the timeline view and the
datepicker labels when selecting annual datasets. The labels should show
only in `yyyy` instead of `MM yyyy` format when annual datasets are the
lowest common denominator selected.

### Notes & Questions About Changes
_{Add additonal notes and outstanding questions here related to changes
in this pull request}_

### Validation / Testing

- Add a dataset with annual time density and verify that the timeline
view labels and the datepicker labels show only the year
- Add datasets with month time density, and verify that now the labels
show the month and the year
- The rest should work as before (date picker shows the appropriate
calendar view)
Copy link

netlify bot commented Aug 20, 2024

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 2faaba0
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/66c4b7595fef9f0008c234ab
😎 Deploy Preview https://deploy-preview-1115--veda-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@slesaad
Copy link
Member

slesaad commented Aug 20, 2024

shouldn't the branches be switched?

hanbyul-here
hanbyul-here previously approved these changes Aug 20, 2024
@hanbyul-here hanbyul-here dismissed their stale review August 20, 2024 15:41

Sorry, I got confused with UI change review

@dzole0311
Copy link
Collaborator Author

@slesaad Sorry, got confused, I should've branched out from the prev tag.

@dzole0311 dzole0311 closed this Aug 20, 2024
@dzole0311 dzole0311 deleted the release/v5.6.1 branch August 20, 2024 15:48
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