Skip to content

Fixed switching the --beta#1291

Closed
MusclePr wants to merge 1 commit into
arkmanager:masterfrom
MusclePr:fix/beta-switching
Closed

Fixed switching the --beta#1291
MusclePr wants to merge 1 commit into
arkmanager:masterfrom
MusclePr:fix/beta-switching

Conversation

@MusclePr

Copy link
Copy Markdown
Contributor

Fixed a bug where the update would not occur even if preaquatica and public were specified alternately with the --beta option.

@MusclePr

MusclePr commented Sep 26, 2025

Copy link
Copy Markdown
Contributor Author

TEST PATTERN

Before:

  1. arkmanager install --beta=preaquatica ... preaquatica
  2. arkmanager update ... preaquatica
  3. arkmanager update --beta=public ... preaquatica
  4. arkmanager update --beta=preaquatica ... preaquatica

After:

  1. arkmanager install --beta=preaquatica ... preaquatica
  2. arkmanager update ... public
  3. arkmanager update --beta=public ... public
  4. arkmanager update --beta=preaquatica ... preaquatica

@klightspeed

klightspeed commented Sep 26, 2025

Copy link
Copy Markdown
Collaborator

If --beta is not being properly passed through to change the current beta, then we need to investigate where it is not passing through. I'm not sure switching back to the public branch is the desired action when no --beta option is provided.

I am not currently at my computer at the moment, and cannot currently have a proper look at this.

@MusclePr

Copy link
Copy Markdown
Contributor Author

Yes, I was worried about that too.

Initially I implemented it like this:

  if [ -n "$appbeta" ]; then
    appbranch="${appbeta}"
  fi

@MusclePr

Copy link
Copy Markdown
Contributor Author

Maybe I'm misunderstanding something fundamental.

Should I use "arkmanager install" instead of "arkmanager update" to switch to the Beta Channel?

@klightspeed

Copy link
Copy Markdown
Collaborator

Could you please check if #1292 works as desired when using `arkmanager update --beta=xxx?

@MusclePr

Copy link
Copy Markdown
Contributor Author

This PR is being closed to be taken over by the owner's PR.

@MusclePr MusclePr closed this Sep 30, 2025
@MusclePr MusclePr deleted the fix/beta-switching branch October 6, 2025 01:07
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