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

gui: Move package to internal. #357

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Sep 12, 2023

This is rebased on #356.

dcrpool is an application and thus all of its packages not intended for external consumers should not be publicly visible.

To that end, this moves the gui package to an internal package meaning that it is no longer a part of the exported main module.

@davecgh davecgh force-pushed the internal_gui branch 2 times, most recently from b69006e to 6980a38 Compare September 13, 2023 07:07
Copy link
Member

@jholdstock jholdstock left a comment

Choose a reason for hiding this comment

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

There is a line in README which needs to be updated

Deploying the user interface requires copying the dcrpool/gui/assets folder from source to a reachable location

dcrpool is an application and thus all of its package not intended for
external consumers should not be publicly visible.

To that end, this moves the gui package to an internal package meaning
that it is no longer a part of the exported main module.
@davecgh
Copy link
Member Author

davecgh commented Sep 13, 2023

There is a line in README which needs to be updated

Deploying the user interface requires copying the dcrpool/gui/assets folder from source to a reachable location

Thanks, updated. As an aside, this should really be changed over to use embed instead imo.

@jholdstock jholdstock merged commit 2392120 into decred:master Sep 13, 2023
2 checks passed
@davecgh davecgh deleted the internal_gui branch September 13, 2023 08:04
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