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.

Sign up for GitHub

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

Jump to bottom

feat: added approximate_user_install_count field to application #1293

Merged
merged 15 commits into from
Oct 19, 2024
Merged

feat: added approximate_user_install_count field to application #1293

merged 15 commits into from
Oct 19, 2024

Conversation

Commandserver
Copy link
Member

It's been a while since my last PR :). Today I've added the missing approximate_user_install_count field to the application class.

Code change checklist

  • I have ensured that all methods and functions are fully documented using doxygen style comments.
  • My code follows the coding style guide.
  • I tested that my change works before raising the PR.
  • I have ensured that I did not break any existing API calls.
  • I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

Commandserver and others added 15 commits May 29, 2023 17:27
@Commandserver
fix: interaction_modal_response::fill_from_json
19a4aa9
@Commandserver
fixed component::fill_from_json which didn't parse some fields
5f76c80
@Commandserver
Merge pull request #86 from brainboxdotcc/dev
8a952bf 
Dev
@Commandserver
feat: added setter methods to dpp::invite class
6139229
@Commandserver
Merge pull request #87 from brainboxdotcc/dev
5b7ca81 
Dev
@Commandserver
feat: added role flag IN_PROMPT
4815cc2
@Commandserver
Merge pull request #93 from brainboxdotcc/dev
b2efb42 
Dev
@Commandserver
Merge pull request #97 from brainboxdotcc/dev
cf5e227 
Dev
@Commandserver
Merge pull request #99 from brainboxdotcc/dev
18c6911 
fix: warning for [=] copy without explkit , rerun coro builder for on…
@Commandserver
Merge pull request #102 from brainboxdotcc/dev
c4a6eee 
Dev
@Commandserver
Merge pull request #103 from brainboxdotcc/dev
a47753a 
Dev
@Commandserver
Merge pull request #105 from brainboxdotcc/dev
717f7e5 
Dev
@Commandserver
Merge pull request #106 from brainboxdotcc/dev
d588861 
Dev
@Commandserver
Merge pull request #107 from brainboxdotcc/dev
20dc2f0 
Dev
@Commandserver
feat: added approximate_user_install_count field to application class
ec7edbc
@netlify Netlify
Copy link

netlify bot commented Oct 18, 2024 •
edited
Loading

✅ Deploy Preview for dpp-dev ready!

Name Link
🔨 Latest commit ec7edbc
🔍 Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/6712c3cfcb6ea10008438424
😎 Deploy Preview https://deploy-preview-1293--dpp-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added documentation Improvements or additions to documentation code Improvements or additions to code. labels Oct 18, 2024
@braindigitalis
Copy link
Contributor

lgtm ship it

@braindigitalis braindigitalis merged commit 73c4751 into brainboxdotcc:dev Oct 19, 2024
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@braindigitalis braindigitalis braindigitalis approved these changes

Assignees
No one assigned
Labels
code Improvements or additions to code. documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Commandserver @braindigitalis