Skip to content

Add local dev command to run backend and plugin#36

Open
dsimansk wants to merge 1 commit into
functions-dev:masterfrom
dsimansk:pr/page-cmd
Open

Add local dev command to run backend and plugin#36
dsimansk wants to merge 1 commit into
functions-dev:masterfrom
dsimansk:pr/page-cmd

Conversation

@dsimansk

Copy link
Copy Markdown
Contributor

Changes

  • Add local page cmd to view

GitHub Pages local development - fetch('plugin.yaml') always fails locally because plugin.yaml is only generated by the CI workflow (helm template). Add a way to test the GitHub Pages site locally (e.g. a script that generates plugin.yaml into the pages directory, or a local dev server setup).

@matejvasek @twoGiants is this somewhat coming closer to the idea stated above?

@matejvasek matejvasek requested a review from twoGiants June 10, 2026 16:58
@dsimansk

Copy link
Copy Markdown
Contributor Author

After spending some time with local dev setup on CRC, I'm rethinking the very simple approach of separate command. I see it needs to be properly wired into local dev loop.

@dsimansk dsimansk changed the title Add local page cmd to view Add local dev command to run backend and plugin Jun 11, 2026

@twoGiants twoGiants left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice, going into the right direction! I'd just move it to the init script. See my comments below.

Comment thread hack/dev.sh

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We're having an init.sh script which starts everything. This script won't work when init was run before, the ports are the same.

Maybe you can wire it all up in the init.sh script, so the pages assets are build every time we start the development environment?

Comment thread README.md
@@ -55,7 +55,9 @@ Available image tags are listed in the [container registry](https://github.com/f
In one terminal window, run:

@twoGiants twoGiants Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you update the entire Option 1 (rename to Setup) section to say that init.sh should be used to start the dev env?

A few more things:

  • Remove the sections:
    • #### Running start-console with Apple silicon and podman
    • Option 2: ...
  • Add a small section in the Readme telling how to view the github pages locally.
  • Add go and helm to the prerequisites.
  • You could make the plugin.yaml generation optional in the init.sh script, then go and helm will also be optional.

Comment thread .gitignore
/bin
pages/plugin.yaml
/public
backend/main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You probably should ignore those too:

  • backend/static/plugin.yaml
  • backend/static/index.html

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, it's leftover from my experiments. I'll clean it up.

Comment thread .gitignore
Comment on lines +56 to +57
/public
backend/main

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Just curious, why are those two added?

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