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.

Sign up for GitHub

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

Jump to bottom

Bring into line with master repo #8

Merged
merged 154 commits into from
Sep 28, 2023
Merged

Bring into line with master repo #8

merged 154 commits into from
Sep 28, 2023

Conversation

ndepaola
Copy link

  • As discussed on discord, pulling these changes will involve switching your database over to Postgres
  • You mentioned your upgrade steps will be:
    1. Make a hard backup of the card DB (just in case) and export_sources to csv
    2. Install postgres create a DB for use with mpcfill
    3. Pull changes, pip install updated requirements.txt
    4. Update ENV with new postgres database info, add GAME env variable (as MTG)
    5. Run migrate
    6. Run collectstatic and rebuild front end
    7. Run import_sources to import from our exported csv
    8. Run update_database
    9. Restart gunicorn and hope site is deployed without issues
  • and I also recommended creating a backup of your VPS image before commencing any of this work

@ndepaola
the pure refactoring commit. bit ugly, needs to be cleaned up.
4563328 
the only change to tests here is wrapping things in a layout
because that's where the change query modal has moved to.
@ndepaola
allow editing individual card names. also a bit rough, need to tidy up.
0550aa2
@ndepaola
autofocus gang
425165d
@ndepaola
add a visual cue to indicate that card names are interactive
82bb76b
@ndepaola
clean up previous commits a bit + refactor most modals into one dir
ca0d6a9
@ndepaola
test changing a single card's query
f9a8eb1
@ndepaola
Merge pull request #162 from chilli-axe/allow-editing-individual-card…
ac96808 
…-names

Allow editing individual card names
@ndepaola
some renaming
cb773ba
@ndepaola
improve readme very slightly + add issue template
929bd0f 
might have to tweak these more later
@ndepaola
fix alignment
792ff09
@ndepaola
fix alignment 2
1ca2570
@ndepaola
handle accents in elasticsearch with asciifolding
094a95a
@ndepaola
incorporate accents into tests
4d9f95e
@ndepaola
update dfc query to accommodate double-faced transforming tokens
853b273
@ndepaola
only provide external link if len > 0
d939ccb
@ndepaola
finish porting across patreon page
1e98b16 
going to deploy this, test against mpcfill.com, then make any
necessary adjustments. can't easily test this locally.
@ndepaola
update snapshot according to accented card name in google drive
417516f
@ndepaola
improvements to patreon modal
2c6b04f
@ndepaola
Merge pull request #164 from chilli-axe/163-accents-not-handled-corre…
7d656ad 
…ctly-by-elasticsearch

handle accents in elasticsearch
@ndepaola
clean up todos
97dfc8b
@ndepaola
forgot to update these fixtures
64ae475
@ndepaola
fuzzy matching to dfc pairs when fuzzy search is enabled
7b5164f 
resolves #165
@ndepaola
allow clearing queries here
89a01f9
@ndepaola
fix loading spinners on contributions page
dc2fb84
@ndepaola
click to copy image identifier
6873c0c
@ndepaola
attempt to smooth out dynamic logo animations
f204ab8 
by waiting until the sample cards query has finished before
commencing the animation
@ndepaola
update snapshots following the previous commit
a7619cc
@ndepaola
fix this type so next builds properly
be81fee
@ndepaola
this ought to start untransformed
60b359c
@ndepaola
and update snapshots
6841476
ndepaola and others added 19 commits September 9, 2023 14:13
@ndepaola
configure sentry
b3aee42 
yes, my sentry DSN is hard-coded into django settings.
i want to track uncaught exceptions in any deployment of this project
through sentry, and sentry reckons the DSN is fine to have publicly
listed: https://docs.sentry.io/product/sentry-basics/dsn-explainer/
@ndepaola
forgot to update these tests
74355bc
@ndepaola
Merge pull request #177 from chilli-axe/tag-hierarchy
f5cb663 
Tag hierarchy
@ndepaola
Merge pull request #178 from chilli-axe/configure-sentry
2bcc3f8 
Configure sentry
@ndepaola
switch to react-dropdown-tree-select from react-multi-select-component
4582425 
still rendering things as a flat list here - maintaining parity
will render tags as a tree view in the next commit
@ndepaola
display tags in tree structure in search settings UI
134d9ed
@ndepaola
describe tag hierarchy in contributions page
e39bda0
@ndepaola
revert displaying tags in card slots
c10ec2f 
i reckon we'll have too many tags on each card for this to be practical
@ndepaola
sort card tags on the way to the frontend
cbd7c2a
@ndepaola
forgot to delete this file
55fd011
@ndepaola
comma gang
d9b8d33
@ndepaola
Merge pull request #179 from chilli-axe/tag-hierarchy-part-2
cccb4b0 
Tag hierarchy part 2
@ndepaola
Merge branch 'master' into clean-up-frontend-todos
cf59f57
@ndepaola
remove unused imports
2cef7fb
@ndepaola
update postgres docker config to work for local development
f2ac03f 
i thought i made this change a little while ago, but it must
have gotten lost in translation
@ndepaola
improve logging slightly
08e7118
@ndepaola
fix visual breakage to grid selector from making modals scrollable
11126b3
@ndepaola
add react windowing libraries
9e789db
@ndepaola
remove default tags that aren't referenced in code anywhere
fe0ce7a 
the intention here is to maximise flexibility for backend deployments
so they can configure tags however they want.
@ndepaola ndepaola mentioned this pull request Sep 27, 2023
Merged
@Investigamer Investigamer merged commit 9212296 into Investigamer:master Sep 28, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ndepaola ndepaola

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ndepaola @Investigamer @glandu-sr @lukegothic