-
Notifications
You must be signed in to change notification settings - Fork 13
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
Ask user to confirm info before creating Install #609
Open
WillNilges
wants to merge
58
commits into
main
Choose a base branch
from
wdn/202
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
fb5d197
fix 500 on bldg
WillNilges 33fb419
Store changed info and print something
WillNilges 4715fce
fancy
WillNilges e35f2ff
Now I gotta update all these tests :(
WillNilges f6e3b5f
This feels very brittle
WillNilges dfaa79c
That's maybe a little better.
WillNilges bbde722
Test cases
WillNilges 07fa3d0
update tests
WillNilges 8367cb6
Add trust_me_bro
WillNilges 2592a49
Add trust_me_bro to test data
WillNilges bdc372e
format
WillNilges af407a4
set trust_me_bro
WillNilges 11515d6
checkpoint
WillNilges 9e873d3
computers
WillNilges ae222c6
Formatting?
WillNilges afbfcd0
Parsed City
WillNilges 8ea2a0b
Remove State + Zip warning
WillNilges 33fcb21
Fix import crash (#598)
Andrew-Dickinson 07377b1
Make name dedupe logic case-insensitive & prefer longer names (#556)
Andrew-Dickinson a3a91b4
Add django-simple-history (#599)
Andrew-Dickinson c89b2fb
Change ticket number to charfield (#562)
Andrew-Dickinson 0ee526c
rename all instances of prod1 -> prod + docs cleanup (#618)
james-otten f125f03
rolling restart stateful sets (#620)
james-otten c4ada29
only backup 'default' (#621)
james-otten 31583aa
Add new encrypted volume (#615)
james-otten 7771b94
use the new one (#623)
james-otten 0014070
Complete volume swap (#616)
james-otten d158dcc
Fix 500 on panorama building save (for real this time) (#622)
WillNilges c050532
Fix: NYC addresses are sent to random New England towns (#604)
Andrew-Dickinson 0ed6ff0
Import fixes (#607)
Andrew-Dickinson a4790c2
Fix: nodes without active installs are hidden on the map (#606)
Andrew-Dickinson 8c32592
Fix: nodes is required to edit building object (#600)
Andrew-Dickinson fa84ad0
Fix: exports get captured by captive nav and don't download (#601)
Andrew-Dickinson 776525d
Add missing assert_ to called_once_with (#619)
Andrew-Dickinson 47d22f6
Fix: crash due to link sheet format change (#626)
Andrew-Dickinson 603772b
Fix: zip code validation only applies to non-NYC addresses (#629)
Andrew-Dickinson 072c27f
got it
WillNilges 17c292d
Update submission format
WillNilges 554590e
get tests working
WillNilges f76fba4
computers
WillNilges 98969e3
lint
WillNilges fd80064
zip
WillNilges 1ec1c46
black
WillNilges 9412847
Check that we say no to NJ properly:
WillNilges 76d3743
move check
WillNilges 2a6bc8f
fix thingie
WillNilges 3db27d4
Add another test
WillNilges 56e5009
Default to str when dumping json object
WillNilges 77177e5
lint
WillNilges 3b5ee9f
Merge branch 'main' into wdn/202
WillNilges da8ad9d
lint
WillNilges 610553e
email
WillNilges 0a58c5e
lint
WillNilges 1e9c1e3
insanity
WillNilges 6e35cb7
email
WillNilges f800b3b
Wild fucking guess, but this test should now create 3 members instead…
WillNilges adcb98a
fix git fuckery
WillNilges af4203f
Test nyc join form but nj zip
WillNilges 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
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.
It's 5AM so I'm making a wild fucking guess, but this should be making 3 instead of 2 now, since phone number is no longer deduped