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

Allow user to sort #1068

Closed
wants to merge 2 commits into from
Closed

Conversation

oleksandr-shvets
Copy link

@oleksandr-shvets oleksandr-shvets commented Feb 16, 2024

See #1013

@oleksandr-shvets
Copy link
Author

@alexanmtz
npm run test failed. Does these tests runs ok on the master branch?

@oleksandr-shvets
Copy link
Author

oleksandr-shvets commented Feb 16, 2024

Screenshot 2024-02-16 at 05 53 24 Codacy Static Code Analysis throws 2 "Unnecessary block" code style issues. It's a bug in Codacy, it doesn't correctly understand ES6 destructuring syntax.

Found related issue

@oleksandr-shvets oleksandr-shvets changed the title Allow user to sort issue #1013 Allow user to sort Feb 16, 2024
@webbdays webbdays mentioned this pull request Feb 20, 2024
6 tasks
@alexanmtz
Copy link
Member

@oleksandr-shvets thanks for working on this and to reporting these issues

@alexanmtz
Copy link
Member

@oleksandr-shvets we use the same listing on /#/profile/tasks/open, and it's not working on that case :(

pageshot of 'Gitpay - Bounties for git issues solved' @ 2024-02-20-1542'57

@oleksandr-shvets
Copy link
Author

@oleksandr-shvets we use the same listing on /#/profile/tasks/open, and it's not working on that case :(

@alexanmtz You are welcome!
Ok, I will do it.

@alexanmtz
Copy link
Member

Thanks @oleksandr-shvets , we merged a stable solution and everything seems to work now, thanks for all your effort and contribution. Feel free to contribute with any other issue you want it

@alexanmtz alexanmtz closed this Apr 10, 2024
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.

None yet

2 participants