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

Update openwrt.md #392

Merged
merged 1 commit into from
Nov 26, 2022
Merged

Update openwrt.md #392

merged 1 commit into from
Nov 26, 2022

Conversation

sgryphon
Copy link
Contributor

Update documentation for latest version of OpenWRT, as per issue #391

Update documentation for latest version of OpenWRT, as per issue #391
@sgryphon sgryphon closed this by deleting the head repository Oct 23, 2022
@ydahhrk
Copy link
Member

ydahhrk commented Oct 24, 2022

Why did you cancel this?

@sgryphon
Copy link
Contributor Author

sgryphon commented Oct 24, 2022

Oops. Sorry, unintentional.

I didn't realise that the fork was auto-created and needed by the change; I thought I had created it accidentally and so cleaned it up (deleted it)... but in retrospect it makes sense that it created the fork (as I can't make the change in the main repo, so it auto-created a fork, and then raised the PR from there).

I'll put it back.

@sgryphon sgryphon reopened this Oct 24, 2022
@sgryphon
Copy link
Contributor Author

Actually, all I needed to do was re-open the PR. (The PR exists independent of the fork it made).

If there are any problems merging, etc, I have the text locally as well.

@ydahhrk ydahhrk merged commit 226b37e into NICMx:master Nov 26, 2022
@ydahhrk
Copy link
Member

ydahhrk commented Nov 26, 2022

I'm terribly sorry I forgot this.

Just tested it. Works fine; I just moved it to the main branch.

@ydahhrk ydahhrk added this to the 4.1.9 milestone Jan 25, 2023
@ydahhrk ydahhrk added the Bug label Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants