This repository has been archived by the owner on Oct 4, 2023. It is now read-only.
[C-2718] Fix mobile sign-up environment issues #3691
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Preview this change https://demo.audius.co/dj-c-2718-mobile-audius-follow |
sliptype
approved these changes
Jul 3, 2023
Comment on lines
+76
to
+91
| let defaultFollowUserIds = new Set([]) | ||
|
|
||
| switch (ENVIRONMENT) { | ||
| case 'production': { | ||
| // user id 51: official audius account | ||
| defaultFollowUserIds = new Set([51]) | ||
| break | ||
| } | ||
| case 'staging': { | ||
| // user id 1964: stage testing account | ||
| defaultFollowUserIds = new Set([1964]) | ||
| break | ||
| } | ||
| } | ||
|
|
||
| return defaultFollowUserIds |
There was a problem hiding this comment.
Maybe a tradeoff in readability, but I think this would be cleaner as an expression. Also removes the need for mutating defaultFollowUserIds!
Suggested change
| let defaultFollowUserIds = new Set([]) | |
| switch (ENVIRONMENT) { | |
| case 'production': { | |
| // user id 51: official audius account | |
| defaultFollowUserIds = new Set([51]) | |
| break | |
| } | |
| case 'staging': { | |
| // user id 1964: stage testing account | |
| defaultFollowUserIds = new Set([1964]) | |
| break | |
| } | |
| } | |
| return defaultFollowUserIds | |
| return new Set({ | |
| // user id 51: official audius account | |
| production: [51], | |
| // user id 1964: stage testing account | |
| staging: [1964], | |
| }[ENVIRONMENT] ?? []) |
There was a problem hiding this comment.
dang i merged and this comment hadnt loaded :/ i agree though
dylanjeffers
added a commit
that referenced
this pull request
Jul 6, 2023
schottra
added a commit
that referenced
this pull request
Jul 7, 2023
* main: (36 commits) [PAY-1566] Adds support for passing color directly to primary button (#3709) [C-2825] Fix push notification registration (#3672) [C-2827] Fix hidden dog ear on search results (#3708) [C-2830] Move storage node selector to common (#3707) [C-1379] Add link to playlist in mobile toast (#3702) [C-2821] Add hidden dog tag to mobile card (#3701) [C-2587] Fix playlist update tooltip (#3700) [QA-560][PAY-1564] Fix hot and new bug (#3703) [PAY-1563] Fix loading into a chat from a push notification (#3699) [C-2478 C-2480] Add suggestedTracks (#3689) Attach ulid package to window (#3697) Fix track screen artist name font weight (#3695) [C-2416] Fix missing hidden playlists on own profile (#3694) Fetch chats on app foreground (#3693) [C-2718] Fix mobile sign-up environment issues (#3691) Update SDK to v3.0.3-beta.63 (#3688) OAuth and Developer Apps QA C-2816 C-2815 C-2794 C-2793 C-2795 C-2797 (#3681) [C-2465] Improve audius-query cache selection performance (#3687) [C-2799] Disallow protected m4p file uploads (#3686) [PAY-1535] Refresh chat messages on render and pull to refresh on chats list (#3673) ...
dylanjeffers
added a commit
that referenced
this pull request
Jul 17, 2023
[cd49430] Manually set the libs state to prevent retry storm (#3753) Michael Piazza [83feb57] Fix android announcement notifs (#3750) Michael Piazza [e375a3f] [PAY-1599] Fix font in Start Conversation prompt (#3748) Reed [9468b70] [C-2851] Add privacy policy to settings page (#3741) Dylan Jeffers [bb67f44] [PAY-1595] Hide chat textinput until chat exists (#3746) Reed [a11cbf1] [PAY-1597] Fix mobile chat screen empty state font (#3744) Reed [f9e5afa] Bump Android again (#3745) Michael Piazza [6d009ae] Get the first storage node for o-auth not all of them (#3743) Andrew Mendelsohn [84bddc0] Bump app versions (#3742) Michael Piazza [c31a97d] Notification cursor reflects clickability (#3740) Reed [de56d62] [C-2823] Improve autogenerated image logic (#3729) Dylan Jeffers [37d76af] [C-1608 C-2750] Fix edit profile/cover photo (#3735) Dylan Jeffers [97368b1] [C-2711] Remove app-store link in force-upgrade for saga (#3737) Dylan Jeffers [9c30287] Pin lerna version (#3738) Sebastian Klingler [21eafd7] Update announcement notification style and hover behavior retry (#3736) Reed [1aca4db] [CON-765] Default v2 signup and fix local EM (#3730) Theo Ilie [b458446] Update announcement notification style and hover behavior (#3733) Reed [c097955] Update announcement notification style and hover behavior (#3733) Reed [e6e59f6] [C-2844] Ensure all tracks fetched on collection page (#3734) Andrew Mendelsohn [be5a951] Add dev storage bootstrap nodes (#3731) Sebastian Klingler [a71526d] [QA-565] Fix notification overflow bug (#3732) Reed [ddb558e] [C-2547] Lineup pagination fixes (#3728) Andrew Mendelsohn [c341156] [C-2679] Track Availability Modal Form (#3720) Andrew Mendelsohn [bacde06] [C-2785] Update collection screen focus effect to fetch lineup (#3722) Kyle Shanks [f870bd2] Make embed not fetch metadata from CN (#3710) Theo Ilie [0fa16c6] [C-2809] Remove user from image hooks (#3723) Dylan Jeffers [ca163ef] [C-2834] Remove replica set usage in web (#3721) Dylan Jeffers [46f2974] [C-2829] Finalize SuggestedTracks (#3706) Dylan Jeffers [8a4b7e8] [PAY-1569] Update blog post link (#3719) Michael Piazza [58070df] Call the DN selection callback in AudiusBackend if using a cached DN (#3718) Marcus Pasell [5e5fdd3] DMs: Add space to learn more text (#3714) Marcus Pasell [1430c7e] DMs: Ensure every permission action has a CTA (#3716) Marcus Pasell [6def75a] Revert "[PAY-1534] Allow Popup to be mounted inside a container (#3669)" (#3715) Marcus Pasell [aa9f49f] DMs: Make links noreferrer noopener (#3713) Marcus Pasell [8bc0a02] [PAY-1463] Fix d to set discovery node (#3690) Marcus Pasell [12fe88d] [PAY-1534] Allow Popup to be mounted inside a container (#3669) Marcus Pasell [a51c83e] [C-2341] Update tracks table to display tracks marked as deleted as deleted (#3712) Kyle Shanks [23ad32e] [PAY-1566] Adds support for passing color directly to primary button (#3709) Randy Schott [79da3ee] [C-2825] Fix push notification registration (#3672) Michelle Brier [475e14e] [C-2827] Fix hidden dog ear on search results (#3708) Dylan Jeffers [946732b] [C-2830] Move storage node selector to common (#3707) Dylan Jeffers [cd0b911] [C-1379] Add link to playlist in mobile toast (#3702) Dylan Jeffers [4059fe8] [C-2821] Add hidden dog tag to mobile card (#3701) Dylan Jeffers [61927c4] [C-2587] Fix playlist update tooltip (#3700) Dylan Jeffers [597f21c] [QA-560][PAY-1564] Fix hot and new bug (#3703) Saliou Diallo [80f72a0] [PAY-1563] Fix loading into a chat from a push notification (#3699) Michael Piazza [ea7e3b1] [C-2478 C-2480] Add suggestedTracks (#3689) Dylan Jeffers [5bb451a] Attach ulid package to window (#3697) Michael Piazza [23e5b24] Fix track screen artist name font weight (#3695) Reed [7de914e] [C-2416] Fix missing hidden playlists on own profile (#3694) Dylan Jeffers [210113d] Fetch chats on app foreground (#3693) Michael Piazza [16ca56a] [C-2718] Fix mobile sign-up environment issues (#3691) Dylan Jeffers [65aa97c] Update SDK to v3.0.3-beta.63 (#3688) Michael Piazza [e8a7c04] OAuth and Developer Apps QA C-2816 C-2815 C-2794 C-2793 C-2795 C-2797 (#3681) nicoback2 [dcd67b9] [C-2465] Improve audius-query cache selection performance (#3687) Dylan Jeffers [05b1940] [C-2799] Disallow protected m4p file uploads (#3686) Andrew Mendelsohn [c099918] [PAY-1535] Refresh chat messages on render and pull to refresh on chats list (#3673) Marcus Pasell [2fccb0b] [CON-765] Default uploads to v2 and remove feature flag (#3679) Theo Ilie
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes sign-up issues stemming from improper use of environment variables that only work on web