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

fix(asana): try another approach to fix customer problem #2365

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

askides
Copy link
Contributor

@askides askides commented Jan 10, 2025

Please remember the Contributing Guidelines ❤️

🌟 What does this PR do?

This PR aim to fix the problem our customer has using Asana integration.

The idea is to try to find another approach to do the same things, adding also some console.debug that could be useful in case for better approach.

I did it completely new from scratch this time dividing the 'MyTasks' page from the 'Projects' page, this way we can be sure that the logic will not get too much complex.

🐛 Recommendations for testing

All changes should be tested across Chrome and Firefox.

Ensure that on asana, you can start the timer from the button, and getting the project and the tags correctly both from any project view and my tasks view.

PS: You can enable debug level logging to see the data resolution directly in console (could be useful to see what our customer see).

📝 Links to relevant issues or information

See this slack thread here.

@askides askides requested review from izontm and with-shrey January 10, 2025 10:06
@askides askides self-assigned this Jan 10, 2025
@askides askides merged commit d120e39 into master Jan 15, 2025
1 check passed
@askides askides deleted the fix/asana-v2 branch January 15, 2025 20:45
@anaraujo
Copy link

LGTM! How can I test the browser extension locally? The guide says to enable development mode but there's no such option in Setttings -> Integrations anymore 🤔

@anaraujo
Copy link

Oops, nevermind @askides , you already merged it 🙈
It would still be nice for me to know how to test things locally so I can help reviewing future PRs ☺️

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