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.

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

Critical fix: make admin non-GET requests work #803

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

ludvigalden
Copy link
Contributor

Hi guys, hope you're well.

This fixes the bug that you have been working on for the past few weeks, or so I heard at least.

To push the changes to admin.utn.se, you need to do the following:

  1. Merge this PR to the development branch.
  2. Merge the development branch with the master branch.
  3. Run ansible-playbook moore in UTNkar/ansible. It takes a while. Make sure it is successful. (You need to configure your environment, SSH user passwords etc., if you haven't done so already. You can find the default password in the repo to log in to with SSH if you have your user defined there.)

If you can't run ansible-playbook moore yourself, recede my admin access on the server and I'll do it for you. I haven't checked if it is even removed, though, but I assume it is.

Sorry about the miss and cheers.

Copy link

@ankurr0y ankurr0y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will merge

@ankurr0y ankurr0y merged commit 88c6509 into UTNkar:development Feb 28, 2024
1 check passed
robindymer added a commit that referenced this pull request Feb 28, 2024
Co-authored-by: Ludvig Aldén <30798446+ludvigalden@users.noreply.github.com>
robindymer added a commit that referenced this pull request Mar 7, 2024
* make admin non-GET requests work

* make admin non-GET requests work (#803)

* use middleware instead of re_path

---------

Co-authored-by: ludvigalden <aldenludvig@gmail.com>
Co-authored-by: Ludvig Aldén <30798446+ludvigalden@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants