Skip to content

feat(nmrs): OpenVpn config expansion#309

Merged
cachebag merged 7 commits intonetworkmanager-rs:dev-openvpnfrom
Dandiggas:feat/add-openvpn-config
Mar 31, 2026
Merged

feat(nmrs): OpenVpn config expansion#309
cachebag merged 7 commits intonetworkmanager-rs:dev-openvpnfrom
Dandiggas:feat/add-openvpn-config

Conversation

@Dandiggas
Copy link
Copy Markdown
Contributor

@Dandiggas Dandiggas commented Mar 26, 2026

Retroactive edit by @cachebag:
Closes #290

Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs Outdated
Comment thread nmrs/src/api/models/vpn.rs Outdated
@cachebag cachebag marked this pull request as draft March 27, 2026 02:34
@Dandiggas Dandiggas force-pushed the feat/add-openvpn-config branch from aba40c4 to 8c4dfd1 Compare March 28, 2026 23:42
@cachebag cachebag force-pushed the feat/add-openvpn-config branch from 05bc9d0 to 4345a74 Compare March 31, 2026 22:19
@cachebag cachebag marked this pull request as ready for review March 31, 2026 22:19
@cachebag cachebag changed the title Feat/add openvpn config feat(nmrs): OpenVpn config expansion Mar 31, 2026
Copy link
Copy Markdown
Collaborator

@cachebag cachebag left a comment

Choose a reason for hiding this comment

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

@Dandiggas Took the liberty to clean the PR up for you. You had it all right just needed some refinements based on my feedback. I was blocked by this on #314 so I figured I'd wrap this up myself.

Let me know if you have any questions or concerns. Thanks for this contribution!

@cachebag cachebag merged commit 4d56fa8 into networkmanager-rs:dev-openvpn Mar 31, 2026
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