-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add Pretendo Support #28
Closed
Closed
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
87a1589
Updated code to use the latest NintendoClients version (+ a few other…
Preloading cc7c79e
Update creation script and add Migration script to support multiple n…
Preloading b467277
left my note to added friends oops
Preloading e1e85c4
realizing that my ideas are stupid, and changing it from a flag to a …
Preloading c39d125
Add args to choose between nintendo & pretendo
Preloading c05e2fd
changes to the migration script & small change to backend
Preloading 8c70cb4
Hotfix: Fix SQL being odd
Preloading fd3ab39
Add broken pretendo backend support, waiting for help on a datetime b…
Preloading 1da19ca
Add pretendo support to the backend uptime counter
Preloading 1ae808a
fix homepage, roster, and active
Preloading 232d317
fix consoles page
Preloading fa60f94
Created the basic "select the network" page
Preloading 5fd14c3
Hotfix: correct and implement some feedback
Preloading 397a4a7
Mostly finish up registering UI and site backend (actually registerin…
Preloading 8ae5da0
You can now add consoles from the menu + maybe fixing the backend for…
Preloading 6f2987e
try to work around missing function on pretendo, currently hangs at r…
Preloading d6e7fac
Add small fixes allowing selecting and deleting consoles
Preloading b8e44b9
Well, the most inelegent way of fixing pretendo support (i guess)
Preloading 628c5ec
Add proper pretendo redirection to the new users and active users (ma…
Preloading 107bb05
Fix console profile redirect
Preloading 681ffff
Make the try profile button work with pretendo
Preloading db89c6f
Finish sidebar, Fix index forgetting about pretendo, and another thing
Preloading beb800f
Done web backend!
Preloading 9145e04
Fix mobile support for the Select Network page
Preloading 998c089
Update template.private.py to be accurate with my private file
Preloading ec164ee
Make discord.py not break and rename some variables in server.py
Preloading f4e569f
Possibly finish off discord.py (open to adding something saying what …
Preloading 6f26c3c
Show what network on the user's page
Preloading f2f9c7a
Add a sign of what network you are on, on the discord rpc
Preloading 12bb21e
Fix API commands to be compatible with before
Preloading bea621d
Remove debug print line
Preloading 1b70062
Uncomment temporary commenting on getPresence (thanks phoenix)
Preloading 51d10c7
Fix mistake where i mixed == with != somehow
Preloading 4918ffb
[Cleanup] Rework __main__: Add cleaner command line & Remove unneeded…
HotaruBlaze 25da282
Merge pull request #1 from HotaruBlaze/Preloading/main
Preloading 8325f52
Fix @HotaruBlaze (Phoenix)'s code
Preloading 5a96a68
Hot... Change?: Add the ability to use -n and --network instead of ju…
Preloading 0eee7ab
Move NetworkIDsToName and other functions to new file + change it to …
Preloading 788ed5d
Resolve some SQL syntax errors (thanks @spotlightishere)
Preloading 909fbeb
Add default to some values in CREATE.sql
Preloading 045b32e
this is actually horrifying, but it fixes pretendo finally.
Preloading a209e53
Add a quick note regarding my hacky fix.
Preloading 40bfcc8
Fix the downloadable client's API, and missing variables
Preloading 431529e
Update copyright & requirements.txt
Preloading File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
from enum import IntEnum | ||
from api.public import nintendoBotFC, pretendoBotFC | ||
|
||
# Selectable networks | ||
class NetworkIDsToName(IntEnum): | ||
nintendo = 0 | ||
pretendo = 1 | ||
|
||
def getBotFriendCodeFromNetworkId(network:int): | ||
match network: | ||
case 0: | ||
return nintendoBotFC | ||
case 1: | ||
return pretendoBotFC | ||
|
||
def nameToNetworkId(network:int): | ||
if network == None: | ||
network = 0 | ||
else: | ||
try: | ||
network = NetworkIDsToName[network].value | ||
except: | ||
network = 0 | ||
return network |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
botFC = str(233790548638).zfill(12) # FC == Friendcode | ||
pretendoBotFC = str(220707964911).zfill(12) # FC == Friendcode | ||
nintendoBotFC = str(439944935945).zfill(12) # FC == Friendcode |
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We made a lot of stability changes and bug fixes a while ago, after this PR was made. This hack can likely be removed, and if the issue still persists then you should open an issue about it on our end. This is obviously bad behavior on our end if it persists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there! A bit ago, I tested, and the program does work without these lines. (yay, thank you!) However, the PR has technically been merged (see 8751376), so I can't modify the PR without causing problems (probably). Why this isn't closed, I do not know. I should probably close it. I'll make sure to try to get the owner of the repo to remove the workaround. Thanks tho!