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: update phoenix_analytics path in Mix.install #3

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

cbuctok
Copy link
Contributor

@cbuctok cbuctok commented Sep 10, 2024

Changed the path of phoenix_analytics from a relative external path to the local directory to ensure proper dependency loading.

Changed the path of phoenix_analytics from a relative external path to the local directory to ensure proper dependency loading.
@lalabuy948
Copy link
Owner

Thank you for your contribution @cbuctok ! Since you are on that topic, could you please add ip: {0, 0, 0, 0} to endpoint_options as at the moment vegeta cannot reach it.

There is a plan to build nice dev environment if this library will pick up.
Using something like: https://github.com/wojtekmach/mix_install_examples/blob/main/phoenix.exs

@cbuctok
Copy link
Contributor Author

cbuctok commented Sep 10, 2024

Thanks for the feedback! I'll go ahead and add ip: {0, 0, 0, 0} to the endpoint_options to ensure compatibility with vegeta. That should help with reaching the endpoint - I use it for my local env but was not sure if it is the way to go.

Add `ip: {0, 0, 0, 0}` to endpoint_options
@@ -1,7 +1,7 @@
#!/usr/bin/env elixir
Mix.install([
{:phoenix_playground, "~> 0.1.5"},
{:phoenix_analytics, path: "../phoenix_analytics", force: true}
{:phoenix_analytics, path: ".", force: true}
Copy link
Owner

Choose a reason for hiding this comment

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

Tip

Nice!

@lalabuy948
Copy link
Owner

lalabuy948 commented Sep 10, 2024

@cbuctok you can target 0.1.2 for this one as well!

@cbuctok cbuctok changed the base branch from master to 0.1.2 September 10, 2024 10:46
@lalabuy948 lalabuy948 merged commit 7da5743 into lalabuy948:0.1.2 Sep 10, 2024
1 check passed
lalabuy948 added a commit that referenced this pull request Sep 10, 2024
* fix: router conflict with error tracker (#4)

* fix: Router conflict with ErrorTracker

Moved `parse_options/2` into `phoenix_analytics_dashboard` macro

---------

Co-authored-by: Greg Evseev <[email protected]>

* version bump

* fix: update phoenix_analytics path in Mix.install (#3)

* fix: update phoenix_analytics path in Mix.install

Changed the path of phoenix_analytics from a relative external path to the local directory to ensure proper dependency loading.

Add `ip: {0, 0, 0, 0}` to endpoint_options

---------

Co-authored-by: Greg Evseev <[email protected]>

---------

Co-authored-by: Greg <[email protected]>
Co-authored-by: Greg Evseev <[email protected]>
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