Skip to content

Conversation

@laemtl
Copy link
Contributor

@laemtl laemtl commented Feb 2, 2021

No description provided.

@laemtl laemtl force-pushed the 2021-02-02-Issue-Trackker-jQuery-cleanup branch from 9c183ca to 386a12b Compare February 2, 2021 19:56
@driusan
Copy link
Collaborator

driusan commented Feb 4, 2021

@maltheism Can you review? Looks good to me but I just skimmed it to be sure it's just doing what it says..

Copy link
Member

@maltheism maltheism left a comment

Choose a reason for hiding this comment

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

Hi @laemtl @driusan, the PR is creating the issue without jquery successfully so I'm approving. I did notice that the redirect is broken after the issue is created. That seems to be on the main branch as well. Lastly, the error message from the server is not being shown and once again that's on main as well.

@maltheism maltheism added the Passed manual tests PR has been successfully tested by at least one peer label Feb 4, 2021
@driusan driusan merged commit 9cc7c5a into aces:main Feb 4, 2021
@laemtl
Copy link
Contributor Author

laemtl commented Feb 4, 2021

@maltheism Thanks for your comment. I believe the same problems were fixed for Publication, after the swal migration. I can have a look.

driusan pushed a commit that referenced this pull request Feb 15, 2021
Small changes following the comment made in #7315

- Fix missing error message
- Fix missing swal callback change (#6195 regression) previously in #7091

Replace #7091
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this pull request Jun 29, 2021
Convert jQuery to fetch API in Issue Tracker.
AlexandraLivadas pushed a commit to AlexandraLivadas/Loris that referenced this pull request Jun 29, 2021
Small changes following the comment made in aces#7315

- Fix missing error message
- Fix missing swal callback change (aces#6195 regression) previously in aces#7091

Replace aces#7091
@ridz1208 ridz1208 added this to the 24.0.0 milestone Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Passed manual tests PR has been successfully tested by at least one peer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants