boneburrito / budgetsimple

MIT License
0 stars 0 forks source link

Update Transaction End Point URL #15

Closed brukeg closed 1 year ago

brukeg commented 1 year ago

The current transaction endpoint urls are: /transaction/api and /transaction/api/<str:id>. We should change that to just /transaction and /transaction/<str:id>

brukeg commented 1 year ago

@AshMudra I think all you have to do is change the paths in transactions/Views.py lines 12 & 13

    path('api/', TransactionView.as_view()),
    path('api/<str:transaction_id>/', TransactionDetailView.as_view()),

but we released another breaking change recently so even getting to this point will take a little bit of doing. You'll need to (roughly):

  1. pull down the most recent changes
  2. create a new database called "budgetsimple"
  3. (optionally) delete the old database
  4. makemigrations and migrate
  5. check to see if you can load the current urls work
  6. make your changes

We can do it together sometime this week.

brukeg commented 1 year ago

@AshMudra path('api/', TransactionView.as_view()), has already been updated by Reed but you can still go for the second URL

path('api/<str:transaction_id>/', TransactionDetailView.as_view()),

it needs to be something like:

path('<str:transaction_id>/', TransactionDetailView.as_view()),