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

New in 2.3: Add Rekordbox post #88

Merged

Conversation

ehendrikd
Copy link
Contributor

@ehendrikd ehendrikd commented May 13, 2020

With screenshots added, this should be ready to merge.

@mixxx-buildbot
Copy link

Can one of the admins verify this patch?

Say: "add to whitelist" to add this user to the whitelist for future PRs.
Say: "ok to test" to start a Jenkins build.
Say: "skip ci" to skip the Jenkins build.
Say: "test this please" to trigger a re-build.

@Be-ing
Copy link
Contributor

Be-ing commented May 13, 2020

add to whitelist

Copy link
Member

@Holzhaus Holzhaus left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

I fixed the markup in some places and added some comments.

pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved
ehendrikd and others added 7 commits May 14, 2020 10:16
Co-authored-by: Jan Holthuis <holthuis.jan@googlemail.com>
Co-authored-by: Jan Holthuis <holthuis.jan@googlemail.com>
Co-authored-by: Jan Holthuis <holthuis.jan@googlemail.com>
Co-authored-by: Jan Holthuis <holthuis.jan@googlemail.com>
Co-authored-by: Jan Holthuis <holthuis.jan@googlemail.com>
@Be-ing
Copy link
Contributor

Be-ing commented Jul 13, 2020

Could you use the full resolution screenshots? 636px × 368px screenshots look quite blurry on my 3840 x 2160 screen.

@ehendrikd ehendrikd marked this pull request as ready for review July 13, 2020 02:25
@Be-ing Be-ing self-requested a review July 13, 2020 02:52
@Be-ing
Copy link
Contributor

Be-ing commented Jul 13, 2020

Looks good to me! @Holzhaus any last comments?

Copy link
Member

@Swiftb0y Swiftb0y left a comment

Choose a reason for hiding this comment

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

There are still two TODO sections we should remove / fill in.

pages/news/2020-XX-XX-new-in-2-3-rekordbox-support.html Outdated Show resolved Hide resolved

<div style="text-align: center;"><img src="{% static '/static/images/news/rekordbox-6-usb.png' %}" /><img src="{% static '/static/images/news/mixxx-rekordbox-usb.png' %}" /></div>

Also note that as with the Serato library feature{# TODO: Add link to serato blogpost #}, we have tried our best to mitigate against certain edge cases of audio files encoded in lossy formats (MP3 and AAC/MP4), where different software's decoders can interpret conflicting timing information, leading to shifted cue points and beatgrids. We hope these are all correct, but some cases may be milliseconds off, and if you find any of these, please let us know on [Zulip](https://mixxx.zulipchat.com).
Copy link
Member

Choose a reason for hiding this comment

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

Also contains a TODO

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Serato news post hasn't been published yet so unfortunately don't have a link to update with.

@Be-ing Be-ing merged commit db75cc1 into mixxxdj:website Jul 14, 2020
@Be-ing
Copy link
Contributor

Be-ing commented Jul 14, 2020

Thanks @ehendrikd!

@Holzhaus
Copy link
Member

Holzhaus commented Jul 14, 2020

Let's not merge all posts at once. We should leave a few days between each of them. Also, the post still containrd some TODOs. Revert?

@Holzhaus
Copy link
Member

Ping

Holzhaus added a commit to Holzhaus/mixxx-website that referenced this pull request Jul 14, 2020
…dbox-removable-devices"

This reverts commit db75cc1, reversing
changes made to 7960a91.
@Holzhaus
Copy link
Member

I revert the merge, would be shame if this post doesn't get the publicity it deserves.

Holzhaus added a commit to Holzhaus/mixxx-website that referenced this pull request Jul 14, 2020
…-3-rekordbox-removable-devices""

This reverts commit 68abb13.
@ehendrikd
Copy link
Contributor Author

The remaining TODO links to the Serato post, so assume it would need to be published first, then this can be updated.

@Be-ing
Copy link
Contributor

Be-ing commented Jul 15, 2020

I don't think there was any need to revert this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
news News posts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants