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

feat(performance): add missing methods (e. g. setAttribute(...), getAttribute(...)) #505

Open
3 of 11 tasks
robingenz opened this issue Dec 6, 2023 · 4 comments · May be fixed by #770
Open
3 of 11 tasks

feat(performance): add missing methods (e. g. setAttribute(...), getAttribute(...)) #505

robingenz opened this issue Dec 6, 2023 · 4 comments · May be fixed by #770
Assignees

Comments

@robingenz
Copy link
Member

robingenz commented Dec 6, 2023

Plugin(s)

  • Analytics
  • App
  • App Check
  • Authentication
  • Crashlytics
  • Cloud Firestore
  • Cloud Messaging
  • Performance
  • Remote Config

Current problem

The Firebase Performance Monitoring plugin currently lacks support for the following methods:

Note: Not all methods are available on all platforms.

Preferred solution

Add support for the missing methods for Android, iOS and Web to the plugin.

Alternative options

No response

Additional context

Before submitting

@triyam
Copy link

triyam commented Dec 6, 2023

/attempt #505

@triyam
Copy link

triyam commented Dec 6, 2023

Let me know when to move ahead

@robingenz
Copy link
Member Author

robingenz commented Dec 7, 2023

@triyam Thank you for your willingness! Unfortunately, I can't assign you the issue as it seems you have little experience in developing Capacitor plugins so far (your changes in definitions.ts does not work for Capacitor). Since I am currently the only maintainer in the Capawesome organization, I don't have the capacity to guide you through the bounty program and conduct intensive reviews. I really hope you understand that!

@triyam
Copy link

triyam commented Dec 7, 2023

Its okay I can understand

ebarooni added a commit that referenced this issue Dec 6, 2024
ebarooni added a commit that referenced this issue Dec 6, 2024
Implement all method definitions on the web.
ebarooni added a commit that referenced this issue Dec 7, 2024
ebarooni added a commit that referenced this issue Dec 7, 2024
Change return of getAttributes on the web.
ebarooni added a commit that referenced this issue Dec 7, 2024
Implement all methods on ios.
ebarooni added a commit that referenced this issue Dec 8, 2024
Add since attribute to definitions.
ebarooni added a commit that referenced this issue Dec 9, 2024
Implement new methods on android.
ebarooni added a commit that referenced this issue Dec 9, 2024
ebarooni added a commit that referenced this issue Dec 10, 2024
Remove classes and update types.
ebarooni added a commit that referenced this issue Dec 11, 2024
Refactor helper methods into separate class.
ebarooni added a commit that referenced this issue Dec 11, 2024
Remove record implementation from android.
ebarooni added a commit that referenced this issue Dec 11, 2024
Remove record implementation from ios.
ebarooni added a commit that referenced this issue Dec 11, 2024
Update method definitions.ts for record.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants