logicalley / website Goto Github PK

When a developer pushes to master, it should automatically trigger a deploy to the staging environment.
However, creating a release with the tag prod-v-* should trigger a deploy to the production environment.

Need to setup testing with jest and @testing-library/react so we can ensure the NExtJS app is properly tested.

Currently, we set a number of cookies mostly from Google Analytics, according to GDPR we need our users to consent to us collecting information about them which is used for analytics.

We should create a cookie consent form similar to that on userwise, it's the perfect one and very simple.

This component should ideally be mounted in the pages/_app.tsx file, so it's going to be available across any page, when the user clicks the accept button, it should set a cookie with the key ANNIE_USER_COOKIE_ACCEPT and the value of 1 to show that the user has accepted the cookie.
You can use this library to serialize and deserialize cookies in NextJS.

Sometimes, we get tracks that have one or two missing platforms, we need a way users can report missing platforms on track page.

Blocked by design from Akachi

The cookie form doesn't display properly on mobile.

Ensure faq link is still available for existing users.
But hyperlink /questions in all places

Good work on the Context Menu. It's cool.

Just a couple of fixes needed for us to close this out:

• Remove the border in the context menu container and switch that to make use of box-shadow instead. You can use the value

box-shadow: -2px 4px 35px rgba(0, 0, 0, 0.25);

• The Copy Link doesn't show a toast notification to notify the user. You can move this method into the utils directory so it can be shared by both TrackDetail and Playlist. Also remember to send analytics for copying the Annie link when Copy Link is clicked as seen here, you don't need to duplicate that whole method since it's just the ANNIE link the user will be copying.

• The Open Link doesn't open the link when clicked, it should open the link in a new tab but it's not working. Also hovering on Open Link doesn't change the text to white as it does for Copy Link.

Currently, when you view a playlist page, if you click on the ellipsis menu beside each track, it opens up the track detail in a new page, the actual behavior we'd like is for a context menu to pop up.

The context menu should give the user the following options:

• Open Link
• Copy Annie Link

NOTE: At any point in time, only one context menu should be open.

You can explore any of the following libraries to achieve this:

• React Contexify
• React Context Menu

I found this article when doing a research about context menus. I recommend react-contexify though because it takes care of a number of things out of the box.

Check out the sentry documentation and reach out to @BolajiOlajide for the Sentry keys!

The last commit broke the CI pipeline because the yarn.lock file wasn't updated.

As discussed via Slack, all you need to do is delete the yarn.lock file , the reinstall the dependencies and everything should be fine.

I believe due to some of the changes I might have changed some of the styling in the footer. Currently in the Track and Playlist pages, the copyright text isn't centralized on the mobile view.

Here's a sample link for you to use to check this bug out.

Since upgrading "next", we've been experiencing slow page loads. I'm not sure if the slow page load is related to the upgrade but it'll be nice to figure out what's causing.

Thread

Netlify is blocking the injected headers in next.config.js so we are unable to have a CSP implementation.

When someone accesses an annie link, it's not always to share a song, so having the title of the modal as Share to ... can be a bit misleading.

Some interfaces contain semi-colons, others don't. This issue ensures all created interface contains a semi-colon.