Merge pull request #89 from bap14/bug/72-dashboard-shows-wrong-version

Make version pull from build context
swiftotter · Oct 6, 2022 · 41e6f5e · 41e6f5e
2 parents ff22025 + 8ed54b4
commit 41e6f5e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/docker/dashboard/Dockerfile b/docker/dashboard/Dockerfile
@@ -1,4 +1,11 @@
+ARG DEN_VERSION="main (development)"
+
 FROM nginx:alpine
+ARG DEN_VERSION
+
+ENV DEN_VERSION=${DEN_VERSION}
 
 ADD index.html /usr/share/nginx/html/index.html
-ADD img/Dnsmasq_icon.svg /usr/share/nginx/html/img/Dnsmasq_icon.svg
+ADD img/Dnsmasq_icon.svg /usr/share/nginx/html/img/Dnsmasq_icon.svg
+
+RUN sed -ie "s/{{DEN_VERSION}}/${DEN_VERSION}/g" /usr/share/nginx/html/index.html
diff --git a/docker/dashboard/index.html b/docker/dashboard/index.html
@@ -83,7 +83,7 @@
 </div><br>
     by Swiftotter<br>
     fork of Warden by David Alger<br>
-    v.in-dev<br>
+    v.{{DEN_VERSION}}<br>
     <a href="https://github.com/swiftotter/den/releases">Check for Updates</a>
 </header>
 <ul class="services-list">
@@ -92,4 +92,4 @@
     <li class="dnsmasq"><a href="https://dnsmasq.den.test/">DNSMasq</a></li>
 </ul>
 </body>
-</html>
+</html>