Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@sillydan1
Copy link
Collaborator

@sillydan1 sillydan1 commented Dec 7, 2022

v1.4.0-rc1 feedback

This PR solves the following problems:

@sillydan1 sillydan1 self-assigned this Dec 7, 2022
@sillydan1 sillydan1 force-pushed the fix/minor-rc-feedbacks branch from 6d70988 to ccae65c Compare December 7, 2022 10:55
@sillydan1 sillydan1 requested a review from Nielswps December 7, 2022 11:05
It messes up the diff for no (good) reason
This also removes the extra boundry check when adding new locations, because it could fail catastrophically in some certain cases
@sillydan1 sillydan1 changed the title Fix: smaller v1.4.0-rc1 feedback issues Fix: v1.4.0-rc1 feedback issues & bugs Dec 8, 2022
Copy link
Contributor

@Nielswps Nielswps left a comment

Choose a reason for hiding this comment

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

Interesting that the shift check is not required, but I guess it should be pretty general for all applications

@sillydan1
Copy link
Collaborator Author

sillydan1 commented Dec 9, 2022

Interesting that the shift check is not required, but I guess it should be pretty general for all applications

Yea 🤔 I was surprised too. I guess that if you need some more custom behavior, you'd just have to work around this behavior...

@sillydan1 sillydan1 requested a review from Nielswps December 9, 2022 19:38
@sillydan1 sillydan1 dismissed Nielswps’s stale review December 9, 2022 19:43

All changes requested have been implemented

@sillydan1 sillydan1 merged commit e9a2989 into master Dec 9, 2022
@sillydan1 sillydan1 deleted the fix/minor-rc-feedbacks branch December 9, 2022 19:43
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