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(github): Fix projects page numElem not passing #1998

Merged
merged 1 commit into from
Nov 23, 2021

Conversation

VagishVela
Copy link
Contributor

This is my first PR in this repo, I read the Contributing.md but let me know if I missed anything!

🌟 What does this PR do?

The current plugin doesn't pass the issue or PR number on GitHub projects through to the toggl button on there. It looks like GitHub might have changed the class the plugin was using to select the element.

After the fix the numElem passes, for example below the #1569 is now being passed:
Screen Shot 2021-08-16 at 8 25 17 AM

🐛 Recommendations for testing

Load the plugin in chrome and firefox, then go to a project page (e.g. a project on this repo) and click a ticket and click the "Start Timer". It should now show the numElem being passed.

📝 Links to relevant issues or information

- Update the selector for numElem so it passes to toggl button
Copy link
Contributor

@nunofmn nunofmn left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 🚀

Thanks for the PR! 😄

@nunofmn nunofmn merged commit 36d68cc into toggl:master Nov 23, 2021
toggl-button-bot added a commit that referenced this pull request Nov 23, 2021
## [1.72.2](1.72.1...1.72.2) (2021-11-23)

### Bug Fixes

* **github:** Fix projects page numElem not passing ([#1998](#1998)) ([36d68cc](36d68cc))
* **zendesk:** Include ticket id in description ([#1996](#1996)) ([c28f1f3](c28f1f3)), closes [#1995](#1995)
@toggl-button-bot
Copy link
Collaborator

🎉 This PR is included in version 1.72.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@VagishVela VagishVela deleted the fix/github-projects-number-passing branch March 2, 2022 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants