Skip to content

V1 docs #111

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

V1 docs #111

wants to merge 5 commits into from

Conversation

shanerbaner82
Copy link

No description provided.

Copy link
Member

@PeteBishwhip PeteBishwhip left a comment

Choose a reason for hiding this comment

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

Just a few comments and questions. Overall looks good. The URL's in deep-links are my biggest concern. Perhaps we should be using example.net instead?

### Laravel
#### For Windows
```shell
set JAVA_HOME=C:\Program Files\Microsoft\jdk-17.0.8.7-hotspot
Copy link
Member

Choose a reason for hiding this comment

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

I'm wondering if this can be replaced with a programatic approach? I'm not sure everyone will have 17.0.8.7 specifically? Not a blocker, just an observation.

Is it possible this isn't required if JAVA_HOME is already set in the Windows Env Variables?


## Private package

To make NativePHP for mobile a reality has taken a lot of work and will continue to require even more. For this reason,
it's not open source and you are not free to distribute or modify its source code.
it's not open source, and you are not free to distribute or modify its source code.

Before you begin, you will need to purchase a license.
Licenses can be obtained via [Anystack](https://checkout.anystack.sh/nativephp-ios).
Copy link
Member

@PeteBishwhip PeteBishwhip Apr 29, 2025

Choose a reason for hiding this comment

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

I believe this link is invalid as @simonhamp is due to merge in the PR to bypass Anystack Billing.

Would be good to have @simonhamp input here.

Choose a reason for hiding this comment

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

Pete is correct here, but I would proceed with this PR as is and I can modify the billing PR to find other checkout.anystack.sh urls throughout the repo. 👍


For example:
```dotenv
https://bifrost.tech/property/456
Copy link
Member

Choose a reason for hiding this comment

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

Where did this URL come from? Accessing it shows "Domain for Sale".

Dialog::alert('Title', 'Message', $buttons, fn ($selected) => {
echo "You selected {$buttons[$selected]}";
});
Dialog::alert('Title', 'Message');
Copy link
Member

Choose a reason for hiding this comment

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

Have we dropped support for buttons on the alerts?

Copy link
Member

@SRWieZ SRWieZ left a comment

Choose a reason for hiding this comment

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

Looking good!

I added some notes here and there ^^


NativePHP for mobile is built to work with Laravel. You can install it into an existing Laravel application, or
[start a new one](https://laravel.com/docs/installation). The most painless way to get PHP and Node up and running on your system is with
[Laravel Herd](https://herd.laravel.com). It's fast and free!

## Private package
Copy link
Member

Choose a reason for hiding this comment

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

I think I will merge the Private package and Install NativePHP for mobile sections.

After running:

```bash
php artisan native:install
Copy link
Member

Choose a reason for hiding this comment

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

Since it references the command and we don't need to run it in this part of the documentation. I will use an inline code block instead of a full code block.

@@ -21,35 +21,35 @@ about whether a particular feature is specific to iOS or Android.

Most of the system-related features are available through the `System` facade.

## Synchronous vs. Asynchronous Methods
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure Synchronous vs. Asynchronous Methods belongs here. Maybe it should have its own page, even if it's a short one.

@@ -58,11 +58,13 @@ You may vibrate the user's device by calling the `vibrate` method:
System::vibrate()
```

_Coming Soon_ Options: `duration` and `intensity`

## Push Notifications
Copy link
Member

Choose a reason for hiding this comment

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

Push notifications should have their own dedicated page, and should contain an installation and configuration guide for FCM and APN.

4. An active [Apple Developer account](https://developer.apple.com/)
5. [A NativePHP for iOS license](https://checkout.anystack.sh/nativephp-ios)
6. _Optional_ iOS device
3. [A NativePHP for mobile license](https://checkout.anystack.sh/nativephp-ios)
Copy link
Contributor

Choose a reason for hiding this comment

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

Possible wrong link also as mentioned by Pete

@@ -81,29 +83,6 @@ if ($token = System::getPushNotificationsToken()) {
Once you have the token, you may use it from your server-based applications to trigger Push Notifications directly to
Copy link
Contributor

Choose a reason for hiding this comment

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

This raised a couple of questions:

  • Should these tokens be stored on the server?
  • Could a user (on the server) have multiple tokens associated with it if they use multiple devices?
  • How to handle tokens when a user logs out and logs back in on with a different account on the device?

Can you give a brief explainer on what's involved to handle messages server side?

@gwleuverink
Copy link
Contributor

Oh boy just a couple more days guys! This looking great already

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.

5 participants