Skip to content

Fixed switching the --beta#1291

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

Fixed switching the --beta#1291
MusclePr wants to merge 1 commit intoarkmanager: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
Copy link
Copy Markdown
Contributor Author

MusclePr commented Sep 26, 2025

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
Copy link
Copy Markdown
Collaborator

klightspeed commented Sep 26, 2025

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