Merge pull request 'Version 3.4' (#78) from 3.4 into main

Reviewed-on: #78

.gitignore

@@ -254,3 +254,5 @@ paket-files/
 
 # Ionide VSCode extension
 .ionide
+
+src/environment.txt

README.md

@@ -6,6 +6,11 @@ Journaling has a long history; it helps people remember what happened, and the a
 
 myPrayerJournal was borne of out of a personal desire [Daniel](https://github.com/danieljsummers) had to have something that would help him with his prayer life. When it's time to pray, it's not really time to use an app, so the design goal here is to keep it simple and unobtrusive. It will also help eliminate some of the downsides to a paper prayer journal, like not remembering whether you've prayed for a request, or running out of room to write another update on one.
 
-## Futher Reading
+## Further Reading
 
 The documentation for the site is at <https://bit-badger.github.io/myPrayerJournal/>.
+
+---
+_Thanks to [JetBrains](https://jb.gg/OpenSource) for licensing their awesome toolset to this project._
+
+[<img src="https://resources.jetbrains.com/storage/products/company/brand/logos/jb_beam.png" alt="JetBrains Logo (Main) logo" width="100" height="100">](https://jb.gg/OpenSource)

publish.ps1

@@ -1,3 +1,3 @@
 #!/snap/bin/pwsh
 Set-Location src/MyPrayerJournal
-dotnet publish -c Release -r linux-x64 -p:PublishSingleFile=true --self-contained false
+dotnet publish -c Release -r linux-x64 -p:PublishSingleFile=true --self-contained false --nologo

src/Dockerfile

@@ -0,0 +1,17 @@
+FROM mcr.microsoft.com/dotnet/sdk:8.0-alpine AS build
+WORKDIR /mpj
+COPY ./MyPrayerJournal/MyPrayerJournal.fsproj ./
+RUN dotnet restore
+
+COPY ./MyPrayerJournal ./
+RUN dotnet publish -c Release -r linux-x64
+RUN rm bin/Release/net8.0/linux-x64/publish/appsettings.*.json || true
+
+FROM  mcr.microsoft.com/dotnet/aspnet:8.0-alpine as final
+WORKDIR /app
+RUN apk add --no-cache icu-libs
+ENV DOTNET_SYSTEM_GLOBALIZATION_INVARIANT=false
+COPY --from=build /mpj/bin/Release/net8.0/linux-x64/publish/ ./
+
+EXPOSE 80
+CMD [ "dotnet", "/app/MyPrayerJournal.dll" ]

src/MyPrayerJournal.Migrate/Migrate.fsproj

@@ -1,22 +0,0 @@
-<Project Sdk="Microsoft.NET.Sdk">
-
-  <PropertyGroup>
-    <OutputType>Exe</OutputType>
-    <TargetFramework>net6.0</TargetFramework>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <Compile Include="Program.fs" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <PackageReference Include="FSharp.Data" Version="4.2.3" />
-    <PackageReference Include="LiteDB" Version="5.0.11" />
-    <PackageReference Include="NodaTime" Version="3.0.9" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <ProjectReference Include="..\MyPrayerJournal\MyPrayerJournal.fsproj" />
-  </ItemGroup>
-
-</Project>

src/MyPrayerJournal.Migrate/Program.fs

@@ -1,57 +0,0 @@
-open FSharp.Data
-open FSharp.Data.CsvExtensions
-open LiteDB
-open MyPrayerJournal.Domain
-open NodaTime
-
-module Subdocs =
-  
-  open FSharp.Data.JsonExtensions
-
-  let history json =
-    match JsonValue.Parse json with
-    | JsonValue.Array hist ->
-      hist
-      |> Array.map (fun h ->
-          { asOf   = (h?asOf.AsInteger64 >> Instant.FromUnixTimeMilliseconds) ()
-            status = h?status.AsString () |> RequestAction.ofString
-            text   = match h?text.AsString () with "" -> None | txt -> Some txt
-            })
-      |> List.ofArray
-    | _ -> []
-  
-  let notes json =
-    match JsonValue.Parse json with
-    | JsonValue.Array notes ->
-        notes
-        |> Array.map (fun n ->
-            { asOf = (n?asOf.AsInteger64 >> Instant.FromUnixTimeMilliseconds) ()
-              notes = n?notes.AsString ()
-              })
-        |> List.ofArray
-    | _ -> []
-
-let oldData = CsvFile.Load("data.csv")
-
-let db = new LiteDatabase("Filename=./mpj.db")
-
-MyPrayerJournal.Data.Startup.ensureDb db
-
-let migrated =
-  oldData.Rows
-  |> Seq.map (fun r ->
-      { id           = r["@id"].Replace ("Requests/", "") |> RequestId.ofString
-        enteredOn    = (r?enteredOn.AsInteger64 >> Instant.FromUnixTimeMilliseconds) ()
-        userId       = UserId r?userId
-        snoozedUntil = (r?snoozedUntil.AsInteger64 >> Instant.FromUnixTimeMilliseconds) ()
-        showAfter    = (r?showAfter.AsInteger64 >> Instant.FromUnixTimeMilliseconds) ()
-        recurType    = r?recurType |> Recurrence.ofString
-        recurCount   = (r?recurCount.AsInteger >> int16) ()
-        history      = Subdocs.history r?history
-        notes        = Subdocs.notes r?notes
-      })
-  |> db.GetCollection<Request>("request").Insert
-
-db.Checkpoint ()
-
-printfn $"Migrated {migrated} requests"

src/MyPrayerJournal.sln

@@ -0,0 +1,26 @@
+
+Microsoft Visual Studio Solution File, Format Version 12.00
+# Visual Studio Version 16
+VisualStudioVersion = 16.0.30114.105
+MinimumVisualStudioVersion = 10.0.40219.1
+Project("{F2A71F9B-5D33-465A-A702-920D77279786}") = "MyPrayerJournal", "MyPrayerJournal\MyPrayerJournal.fsproj", "{6BD5A3C8-F859-42A0-ACD7-A5819385E828}"
+EndProject
+Global
+	GlobalSection(SolutionConfigurationPlatforms) = preSolution
+		Debug|Any CPU = Debug|Any CPU
+		Release|Any CPU = Release|Any CPU
+	EndGlobalSection
+	GlobalSection(SolutionProperties) = preSolution
+		HideSolutionNode = FALSE
+	EndGlobalSection
+	GlobalSection(ProjectConfigurationPlatforms) = postSolution
+		{6BD5A3C8-F859-42A0-ACD7-A5819385E828}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{6BD5A3C8-F859-42A0-ACD7-A5819385E828}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{6BD5A3C8-F859-42A0-ACD7-A5819385E828}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{6BD5A3C8-F859-42A0-ACD7-A5819385E828}.Release|Any CPU.Build.0 = Release|Any CPU
+		{72B57736-8721-4636-A309-49FA4222416E}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{72B57736-8721-4636-A309-49FA4222416E}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{72B57736-8721-4636-A309-49FA4222416E}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{72B57736-8721-4636-A309-49FA4222416E}.Release|Any CPU.Build.0 = Release|Any CPU
+	EndGlobalSection
+EndGlobal

src/MyPrayerJournal/.gitignore

@@ -1,5 +1,2 @@
-## LiteDB database file
-*.db
-
 ## Development settings
 appsettings.Development.json

src/MyPrayerJournal/Data.fs

@@ -1,209 +1,202 @@
-module MyPrayerJournal.Data
+module MyPrayerJournal.Data
 
-open LiteDB
-open NodaTime
-open System
-open System.Threading.Tasks
+/// Table(!) used by myPrayerJournal
+module Table =
 
-// fsharplint:disable MemberNames
-
-/// LiteDB extensions
-[<AutoOpen>]
-module Extensions =
-  
-  /// Extensions on the LiteDatabase class
-  type LiteDatabase with
-    /// The Request collection
-    member this.requests
-      with get () = this.GetCollection<Request> "request"
-    /// Async version of the checkpoint command (flushes log)
-    member this.saveChanges () =
-      this.Checkpoint ()
-      Task.CompletedTask
+    /// Requests
+    [<Literal>]
+    let Request = "mpj.request"
 
 
-/// Map domain to LiteDB
-//  It does mapping, but since we're so DU-heavy, this gives us control over the JSON representation
+/// JSON serialization customizations
 [<RequireQualifiedAccess>]
-module Mapping =
+module Json =
 
-  /// Map a history entry to BSON
-  let historyToBson (hist : History) : BsonValue =
-    let doc = BsonDocument ()
-    doc["asOf"]   <- hist.asOf.ToUnixTimeMilliseconds ()
-    doc["status"] <- RequestAction.toString hist.status
-    doc["text"]   <- match hist.text with Some t -> t | None -> ""
-    upcast doc
+    open System.Text.Json.Serialization
 
-  /// Map a BSON document to a history entry
-  let historyFromBson (doc : BsonValue) =
-    { asOf   = Instant.FromUnixTimeMilliseconds doc["asOf"].AsInt64
-      status = RequestAction.ofString doc["status"].AsString
-      text   = match doc["text"].AsString with "" -> None | txt -> Some txt
+    /// Convert a wrapped DU to/from its string representation
+    type WrappedJsonConverter<'T>(wrap : string -> 'T, unwrap : 'T -> string) =
+        inherit JsonConverter<'T>()
+        override _.Read(reader, _, _) =
+            wrap (reader.GetString()) 
+        override _.Write(writer, value, _) =
+            writer.WriteStringValue(unwrap value)
+    
+    open System.Text.Json
+    open NodaTime.Serialization.SystemTextJson
+
+    /// JSON serializer options to support the target domain
+    let options =
+        let opts = JsonSerializerOptions()
+        [ WrappedJsonConverter(Recurrence.ofString,    Recurrence.toString) :> JsonConverter
+          WrappedJsonConverter(RequestAction.ofString, RequestAction.toString)
+          WrappedJsonConverter(RequestId.ofString,     RequestId.toString)
+          WrappedJsonConverter(UserId,                 UserId.toString)
+          JsonFSharpConverter() ]
+        |> List.iter opts.Converters.Add
+        let _ = opts.ConfigureForNodaTime NodaTime.DateTimeZoneProviders.Tzdb
+        opts.PropertyNamingPolicy   <- JsonNamingPolicy.CamelCase
+        opts.DefaultIgnoreCondition <- JsonIgnoreCondition.WhenWritingNull
+        opts
+
+
+open BitBadger.Documents.Postgres
+
+/// Connection 
+[<RequireQualifiedAccess>]
+module Connection =
+
+    open BitBadger.Documents
+    open Microsoft.Extensions.Configuration
+    open Npgsql
+    open System.Text.Json
+
+    /// Ensure the database is ready to use
+    let private ensureDb () = backgroundTask {
+        do! Custom.nonQuery "CREATE SCHEMA IF NOT EXISTS mpj" []
+        do! Definition.ensureTable         Table.Request
+        do! Definition.ensureDocumentIndex Table.Request Optimized
     }
 
-  /// Map a note entry to BSON
-  let noteToBson (note : Note) : BsonValue =
-    let doc = BsonDocument ()
-    doc["asOf"]  <- note.asOf.ToUnixTimeMilliseconds ()
-    doc["notes"] <- note.notes
-    upcast doc
-
-  /// Map a BSON document to a note entry
-  let noteFromBson (doc : BsonValue) =
-    { asOf  = Instant.FromUnixTimeMilliseconds doc["asOf"].AsInt64
-      notes = doc["notes"].AsString
+    /// Set up the data environment
+    let setUp (cfg : IConfiguration) = backgroundTask {
+        let builder = NpgsqlDataSourceBuilder (cfg.GetConnectionString "mpj")
+        let _ = builder.UseNodaTime()
+        Configuration.useDataSource (builder.Build())
+        Configuration.useIdField "id"
+        Configuration.useSerializer
+            { new IDocumentSerializer with
+                member _.Serialize<'T>(it : 'T)       = JsonSerializer.Serialize(it, Json.options)
+                member _.Deserialize<'T>(it : string) = JsonSerializer.Deserialize<'T>(it, Json.options)
+            }
+        do! ensureDb ()
     }
 
-  /// Map a request to its BSON representation
-  let requestToBson req : BsonValue =
-    let doc = BsonDocument ()
-    doc["_id"]          <- RequestId.toString req.id
-    doc["enteredOn"]    <- req.enteredOn.ToUnixTimeMilliseconds ()
-    doc["userId"]       <- UserId.toString req.userId
-    doc["snoozedUntil"] <- req.snoozedUntil.ToUnixTimeMilliseconds ()
-    doc["showAfter"]    <- req.showAfter.ToUnixTimeMilliseconds ()
-    doc["recurType"]    <- Recurrence.toString req.recurType
-    doc["recurCount"]   <- BsonValue req.recurCount
-    doc["history"]      <- BsonArray (req.history |> List.map historyToBson |> Seq.ofList)
-    doc["notes"]        <- BsonArray (req.notes   |> List.map noteToBson    |> Seq.ofList)
-    upcast doc
 
-  /// Map a BSON document to a request
-  let requestFromBson (doc : BsonValue) =
-    { id           = RequestId.ofString doc["_id"].AsString
-      enteredOn    = Instant.FromUnixTimeMilliseconds doc["enteredOn"].AsInt64
-      userId       = UserId doc["userId"].AsString
-      snoozedUntil = Instant.FromUnixTimeMilliseconds doc["snoozedUntil"].AsInt64
-      showAfter    = Instant.FromUnixTimeMilliseconds doc["showAfter"].AsInt64
-      recurType    = Recurrence.ofString doc["recurType"].AsString
-      recurCount   = int16 doc["recurCount"].AsInt32
-      history      = doc["history"].AsArray |> Seq.map historyFromBson |> List.ofSeq
-      notes        = doc["notes"].AsArray   |> Seq.map noteFromBson    |> List.ofSeq
-      }
+/// Data access functions for requests
+[<RequireQualifiedAccess>]
+module Request =
     
-  /// Set up the mapping
-  let register () = 
-    BsonMapper.Global.RegisterType<Request>(
-      Func<Request, BsonValue> requestToBson, Func<BsonValue, Request> requestFromBson)
-
-/// Code to be run at startup
-module Startup =
-  
-  /// Ensure the database is set up
-  let ensureDb (db : LiteDatabase) =
-    db.requests.EnsureIndex (fun it -> it.userId) |> ignore
-    Mapping.register ()
-
-
-/// Async wrappers for LiteDB, and request -> journal mappings
-[<AutoOpen>]
-module private Helpers =
-
-  open System.Linq
-
-  /// Convert a sequence to a list asynchronously (used for LiteDB IO)
-  let toListAsync<'T> (q : 'T seq) =
-    (q.ToList >> Task.FromResult) ()
-
-  /// Convert a sequence to a list asynchronously (used for LiteDB IO)
-  let firstAsync<'T> (q : 'T seq) =
-    q.FirstOrDefault () |> Task.FromResult
-
-  /// Async wrapper around a request update
-  let doUpdate (db : LiteDatabase) (req : Request) =
-    db.requests.Update req |> ignore
-    Task.CompletedTask
-
-
-/// Retrieve a request, including its history and notes, by its ID and user ID
-let tryFullRequestById reqId userId (db : LiteDatabase) = backgroundTask {
-  let! req = db.requests.Find (Query.EQ ("_id", RequestId.toString reqId)) |> firstAsync
-  return match box req with null -> None | _ when req.userId = userId -> Some req | _ -> None
-  }
-
-/// Add a history entry
-let addHistory reqId userId hist db = backgroundTask {
-  match! tryFullRequestById reqId userId db with
-  | Some req -> do! doUpdate db { req with history = hist :: req.history }
-  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-  }
-
-/// Add a note
-let addNote reqId userId note db = backgroundTask {
-  match! tryFullRequestById reqId userId db with
-  | Some req -> do! doUpdate db { req with notes = note :: req.notes }
-  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-  }
+    open NodaTime
 
     /// Add a request
-let addRequest (req : Request) (db : LiteDatabase) =
-  db.requests.Insert req |> ignore
+    let add req =
+        insert<Request> Table.Request req
 
-// FIXME: make a common function here
+    /// Does a request exist for the given request ID and user ID?
+    let existsById (reqId : RequestId) (userId : UserId) =
+        Exists.byContains Table.Request {| Id = reqId; UserId = userId |}
     
-/// Retrieve all answered requests for the given user
-let answeredRequests userId (db : LiteDatabase) = backgroundTask {
-  let! reqs = db.requests.Find (Query.EQ ("userId", UserId.toString userId)) |> toListAsync
+    /// Retrieve a request by its ID and user ID
+    let tryById reqId userId = backgroundTask {
+        match! Find.byId<string, Request> Table.Request (RequestId.toString reqId) with
+        | Some req when req.UserId = userId -> return Some req
+        | _ -> return None
+    }
+    
+    /// Update recurrence for a request
+    let updateRecurrence reqId userId (recurType : Recurrence) = backgroundTask {
+        let dbId = RequestId.toString reqId
+        match! existsById reqId userId with
+        | true -> do! Patch.byId Table.Request dbId {| Recurrence = recurType |}
+        | false -> invalidOp $"Request ID {dbId} not found"
+    }
+
+    /// Update the show-after time for a request
+    let updateShowAfter reqId userId (showAfter : Instant option) = backgroundTask {
+        let dbId = RequestId.toString reqId
+        match! existsById reqId userId with
+        | true -> do! Patch.byId Table.Request dbId {| ShowAfter = showAfter |}
+        | false -> invalidOp $"Request ID {dbId} not found"
+    }
+
+    /// Update the snoozed and show-after values for a request
+    let updateSnoozed reqId userId (until : Instant option) = backgroundTask {
+        let dbId = RequestId.toString reqId
+        match! existsById reqId userId with
+        | true -> do! Patch.byId Table.Request dbId {| SnoozedUntil = until; ShowAfter = until |}
+        | false -> invalidOp $"Request ID {dbId} not found"
+    }
+
+
+/// Specific manipulation of history entries
+[<RequireQualifiedAccess>]
+module History =
+
+    /// Add a history entry
+    let add reqId userId hist = backgroundTask {
+        let dbId = RequestId.toString reqId
+        match! Request.tryById reqId userId with
+        | Some req ->
+            do! Patch.byId Table.Request dbId {| History = (hist :: req.History) |> List.sortByDescending (_.AsOf) |}
+        | None -> invalidOp $"Request ID {dbId} not found"
+    }
+
+
+/// Data access functions for journal-style requests
+[<RequireQualifiedAccess>]
+module Journal =
+
+    /// Retrieve a user's answered requests
+    let answered (userId : UserId) = backgroundTask {
+        let! reqs =
+            Custom.list
+                $"""{Query.Find.byContains Table.Request} AND {Query.whereJsonPathMatches "@stat"}"""
+                [ jsonParam "@criteria" {| UserId = userId |}
+                  "@stat", Sql.string """$.history[0].status ? (@ == "Answered")""" ]
+                fromData<Request>
         return
             reqs
-    |> Seq.map JournalRequest.ofRequestFull
-    |> Seq.filter (fun it -> it.lastStatus = Answered)
-    |> Seq.sortByDescending (fun it -> it.asOf)
-    |> List.ofSeq
-  }
-
-/// Retrieve the user's current journal
-let journalByUserId userId (db : LiteDatabase) = backgroundTask {
-  let! jrnl = db.requests.Find (Query.EQ ("userId", UserId.toString userId)) |> toListAsync
-  return
-    jrnl
+            |> Seq.ofList
             |> Seq.map JournalRequest.ofRequestLite
-    |> Seq.filter (fun it -> it.lastStatus <> Answered)
-    |> Seq.sortBy (fun it -> it.asOf)
+            |> Seq.filter (fun it -> it.LastStatus = Answered)
+            |> Seq.sortByDescending (_.AsOf)
             |> List.ofSeq
     }
 
-/// Does the user have any snoozed requests?
-let hasSnoozed userId now (db : LiteDatabase) = backgroundTask {
-  let! jrnl = journalByUserId userId db
-  return jrnl |> List.exists (fun r -> r.snoozedUntil > now)
+    /// Retrieve a user's current prayer journal (includes snoozed and non-immediate recurrence)
+    let forUser (userId : UserId) = backgroundTask {
+        let! reqs =
+            Custom.list
+                $"""{Query.Find.byContains Table.Request} AND {Query.whereJsonPathMatches "@stat"}"""
+                [ jsonParam "@criteria" {| UserId = userId |}
+                  "@stat", Sql.string """$.history[0].status ? (@ <> "Answered")""" ]
+                fromData<Request>
+        return
+            reqs
+            |> Seq.ofList
+            |> Seq.map JournalRequest.ofRequestLite
+            |> Seq.filter (fun it -> it.LastStatus <> Answered)
+            |> Seq.sortBy (_.AsOf)
+            |> List.ofSeq
     }
 
-/// Retrieve a request by its ID and user ID (without notes and history)
-let tryRequestById reqId userId db = backgroundTask {
-  let! req = tryFullRequestById reqId userId db
-  return req |> Option.map (fun r -> { r with history = []; notes = [] })
+    /// Does the user's journal have any snoozed requests?
+    let hasSnoozed userId now = backgroundTask {
+        let! jrnl = forUser userId
+        return jrnl |> List.exists (fun r -> defaultArg (r.SnoozedUntil |> Option.map (fun it -> it > now)) false)
     }
 
-/// Retrieve notes for a request by its ID and user ID
-let notesById reqId userId (db : LiteDatabase) = backgroundTask {
-  match! tryFullRequestById reqId userId db with | Some req -> return req.notes | None -> return []
-  }
-    
-/// Retrieve a journal request by its ID and user ID
-let tryJournalById reqId userId (db : LiteDatabase) = backgroundTask {
-  let! req = tryFullRequestById reqId userId db
+    let tryById reqId userId = backgroundTask {
+        let! req = Request.tryById reqId userId
         return req |> Option.map JournalRequest.ofRequestLite
     }
 
-/// Update the recurrence for a request
-let updateRecurrence reqId userId recurType recurCount db = backgroundTask {
-  match! tryFullRequestById reqId userId db with
-  | Some req -> do! doUpdate db { req with recurType = recurType; recurCount = recurCount }
-  | None     -> invalidOp $"{RequestId.toString reqId} not found"
+
+/// Specific manipulation of note entries
+[<RequireQualifiedAccess>]
+module Note =
+
+    /// Add a note
+    let add reqId userId note = backgroundTask {
+        let dbId = RequestId.toString reqId
+        match! Request.tryById reqId userId with
+        | Some req ->
+            do! Patch.byId Table.Request dbId {| Notes = (note :: req.Notes) |> List.sortByDescending (_.AsOf) |}
+        | None -> invalidOp $"Request ID {dbId} not found"
     }
 
-/// Update a snoozed request
-let updateSnoozed reqId userId until db = backgroundTask {
-  match! tryFullRequestById reqId userId db with
-  | Some req -> do! doUpdate db { req with snoozedUntil = until; showAfter = until }
-  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-  }
-
-/// Update the "show after" timestamp for a request
-let updateShowAfter reqId userId showAfter db = backgroundTask {
-  match! tryFullRequestById reqId userId db with
-  | Some req -> do! doUpdate db { req with showAfter = showAfter }
-  | None     -> invalidOp $"{RequestId.toString reqId} not found"
+    /// Retrieve notes for a request by the request ID
+    let byRequestId reqId userId = backgroundTask {
+        match! Request.tryById reqId userId with Some req -> return req.Notes | None -> return []
     }

src/MyPrayerJournal/Dates.fs

@@ -46,16 +46,14 @@ let twoMonths   = 86_400.
 
 open System
 
-/// Convert from a JavaScript "ticks" value to a date/time
-let fromJs ticks = DateTime.UnixEpoch + TimeSpan.FromTicks (ticks * 10_000L)
-
-let formatDistance (startDate : Instant) (endDate : Instant) =
+/// Format the distance between two instants in approximate English terms
+let formatDistance (startOn : Instant) (endOn : Instant) =
     let format (token, number) locale =
         let labels = locales |> Map.find locale
         match number with 1 -> fst labels[token] | _ -> sprintf (snd labels[token]) number
     let round (it : float) = Math.Round it |> int
 
-  let diff        = startDate - endDate
+    let diff        = startOn - endOn
     let minutes     = Math.Abs diff.TotalMinutes
     let formatToken =
         let months = minutes / aMonth |> round
@@ -74,5 +72,5 @@ let formatDistance (startDate : Instant) (endDate : Instant) =
         | _                            -> AlmostXYears, years + 1
   
     format formatToken "en-US"
-  |> match startDate > endDate with true -> sprintf "%s ago" | false -> sprintf "in %s"
+    |> match startOn > endOn with true -> sprintf "%s ago" | false -> sprintf "in %s"
 

src/MyPrayerJournal/Domain.fs

@@ -1,67 +1,84 @@
-[<AutoOpen>]
 /// The data model for myPrayerJournal
+[<AutoOpen>]
 module MyPrayerJournal.Domain
 
-// fsharplint:disable RecordFieldNames
-
+open System
 open Cuid
 open NodaTime
 
 /// An identifier for a request
-type RequestId =
-  | RequestId of Cuid
+type RequestId = RequestId of Cuid
 
 /// Functions to manipulate request IDs
 module RequestId =
+    
     /// The string representation of the request ID
     let toString = function RequestId x -> Cuid.toString x
+    
     /// Create a request ID from a string representation
     let ofString = Cuid >> RequestId
 
 
 /// The identifier of a user (the "sub" part of the JWT)
-type UserId =
-  | UserId of string
+type UserId = UserId of string
 
 /// Functions to manipulate user IDs
 module UserId =
+    
     /// The string representation of the user ID
     let toString = function UserId x -> x
 
 
 /// How frequently a request should reappear after it is marked "Prayed"
 type Recurrence =
+    
+    /// A request should reappear immediately at the bottom of the list
     | Immediate
-  | Hours
-  | Days
-  | Weeks
+    
+    /// A request should reappear in the given number of hours
+    | Hours of int16
+    
+    /// A request should reappear in the given number of days
+    | Days  of int16
+    
+    /// A request should reappear in the given number of weeks (7-day increments)
+    | Weeks of int16
 
 /// Functions to manipulate recurrences
 module Recurrence =
+    
     /// Create a string representation of a recurrence
     let toString =
         function
         | Immediate -> "Immediate"
-    | Hours     -> "Hours"
-    | Days      -> "Days"
-    | Weeks     -> "Weeks"
+        | Hours   h -> $"{h} Hours"
+        | Days    d -> $"{d} Days"
+        | Weeks   w -> $"{w} Weeks"
+    
     /// Create a recurrence value from a string
     let ofString =
         function
         | "Immediate" -> Immediate
-    | "Hours"     -> Hours
-    | "Days"      -> Days
-    | "Weeks"     -> Weeks
+        | it when it.Contains " " ->
+            let parts = it.Split " "
+            let length = Convert.ToInt16 parts[0]
+            match parts[1] with
+            | "Hours" -> Hours length
+            | "Days"  -> Days  length
+            | "Weeks" -> Weeks length
+            | _       -> invalidOp $"{parts[1]} is not a valid recurrence"
         | it -> invalidOp $"{it} is not a valid recurrence"
+    
     /// An hour's worth of seconds
     let private oneHour = 3_600L
+    
     /// The duration of the recurrence (in milliseconds)
-  let duration x =
-    (match x with
+    let duration =
+        function
         | Immediate -> 0L
-    | Hours     -> oneHour
-    | Days      -> oneHour * 24L
-    | Weeks     -> oneHour * 24L * 7L)
+        | Hours   h -> int64 h * oneHour
+        | Days    d -> int64 d * oneHour * 24L
+        | Weeks   w -> int64 w * oneHour * 24L * 7L
 
 
 /// The action taken on a request as part of a history entry
@@ -71,125 +88,9 @@ type RequestAction =
     | Updated
     | Answered
 
-
-/// History is a record of action taken on a prayer request, including updates to its text
-[<CLIMutable; NoComparison; NoEquality>]
-type History = {
-  /// The time when this history entry was made
-  asOf   : Instant
-  /// The status for this history entry
-  status : RequestAction
-  /// The text of the update, if applicable
-  text   : string option
-  }
-
-/// Note is a note regarding a prayer request that does not result in an update to its text
-[<CLIMutable; NoComparison; NoEquality>]
-type Note = {
-  /// The time when this note was made
-  asOf  : Instant
-  /// The text of the notes
-  notes : string
-  }
-
-/// Request is the identifying record for a prayer request
-[<CLIMutable; NoComparison; NoEquality>]
-type Request = {
-  /// The ID of the request
-  id           : RequestId
-  /// The time this request was initially entered
-  enteredOn    : Instant
-  /// The ID of the user to whom this request belongs ("sub" from the JWT)
-  userId       : UserId
-  /// The time at which this request should reappear in the user's journal by manual user choice
-  snoozedUntil : Instant
-  /// The time at which this request should reappear in the user's journal by recurrence
-  showAfter    : Instant
-  /// The type of recurrence for this request
-  recurType    : Recurrence
-  /// How many of the recurrence intervals should occur between appearances in the journal
-  recurCount   : int16
-  /// The history entries for this request
-  history      : History list
-  /// The notes for this request
-  notes        : Note list
-  }
-with
-  /// An empty request
-  static member empty =
-    { id           = Cuid.generate () |> RequestId
-      enteredOn    = Instant.MinValue
-      userId       = UserId ""
-      snoozedUntil = Instant.MinValue
-      showAfter    = Instant.MinValue
-      recurType    = Immediate
-      recurCount   = 0s
-      history      = []
-      notes        = []
-      }
-
-/// JournalRequest is the form of a prayer request returned for the request journal display. It also contains
-/// properties that may be filled for history and notes.
-[<NoComparison; NoEquality>]
-type JournalRequest = {
-  /// The ID of the request (just the CUID part)
-  requestId    : RequestId
-  /// The ID of the user to whom the request belongs
-  userId       : UserId
-  /// The current text of the request
-  text         : string
-  /// The last time action was taken on the request
-  asOf         : Instant
-  /// The last status for the request
-  lastStatus   : RequestAction
-  /// The time that this request should reappear in the user's journal
-  snoozedUntil : Instant
-  /// The time after which this request should reappear in the user's journal by configured recurrence
-  showAfter    : Instant
-  /// The type of recurrence for this request
-  recurType    : Recurrence
-  /// How many of the recurrence intervals should occur between appearances in the journal
-  recurCount   : int16
-  /// History entries for the request
-  history      : History list
-  /// Note entries for the request
-  notes        : Note list
-  }
-
-/// Functions to manipulate journal requests
-module JournalRequest =
-
-  /// Convert a request to the form used for the journal (precomputed values, no notes or history)
-  let ofRequestLite (req : Request) =
-    let hist = req.history |> List.sortByDescending (fun it -> it.asOf) |> List.tryHead
-    { requestId    = req.id
-      userId       = req.userId
-      text         = req.history
-                      |> List.filter (fun it -> Option.isSome it.text)
-                      |> List.sortByDescending (fun it -> it.asOf)
-                      |> List.tryHead
-                      |> Option.map (fun h -> Option.get h.text)
-                      |> Option.defaultValue ""
-      asOf         = match hist with Some h -> h.asOf   | None -> Instant.MinValue
-      lastStatus   = match hist with Some h -> h.status | None -> Created
-      snoozedUntil = req.snoozedUntil
-      showAfter    = req.showAfter
-      recurType    = req.recurType
-      recurCount   = req.recurCount
-      history      = []
-      notes        = []
-      }
-
-  /// Same as `ofRequestLite`, but with notes and history
-  let ofRequestFull req =
-    { ofRequestLite req with 
-        history = req.history
-        notes   = req.notes
-      }
-
-
 /// Functions to manipulate request actions
 module RequestAction =
+  
     /// Create a string representation of an action
     let toString =
         function
@@ -197,6 +98,7 @@ module RequestAction =
         | Prayed   -> "Prayed"
         | Updated  -> "Updated"
         | Answered -> "Answered"
+    
     /// Create a RequestAction from a string
     let ofString =
         function
@@ -205,9 +107,173 @@ module RequestAction =
         | "Updated"  -> Updated
         | "Answered" -> Answered
         | it         -> invalidOp $"Bad request action {it}"
+    
+
+
+/// History is a record of action taken on a prayer request, including updates to its text
+[<CLIMutable; NoComparison; NoEquality>]
+type History =
+    {   /// The time when this history entry was made
+        AsOf : Instant
+        
+        /// The status for this history entry
+        Status : RequestAction
+        
+        /// The text of the update, if applicable
+        Text : string option
+    }
+
+/// Functions to manipulate history entries
+module History =
+  
     /// Determine if a history's status is `Created`
-  let isCreated hist = hist.status = Created
+    let isCreated hist = hist.Status = Created
+    
     /// Determine if a history's status is `Prayed`
-  let isPrayed hist = hist.status = Prayed
+    let isPrayed hist = hist.Status = Prayed
+    
     /// Determine if a history's status is `Answered`
-  let isAnswered hist = hist.status = Answered
+    let isAnswered hist = hist.Status = Answered
+
+
+/// Note is a note regarding a prayer request that does not result in an update to its text
+[<CLIMutable; NoComparison; NoEquality>]
+type Note =
+    {   /// The time when this note was made
+        AsOf : Instant
+        
+        /// The text of the notes
+        Notes : string
+    }
+
+
+/// Request is the identifying record for a prayer request
+[<CLIMutable; NoComparison; NoEquality>]
+type Request =
+    {   /// The ID of the request
+        Id : RequestId
+        
+        /// The time this request was initially entered
+        EnteredOn : Instant
+        
+        /// The ID of the user to whom this request belongs ("sub" from the JWT)
+        UserId : UserId
+        
+        /// The time at which this request should reappear in the user's journal by manual user choice
+        SnoozedUntil : Instant option
+        
+        /// The time at which this request should reappear in the user's journal by recurrence
+        ShowAfter : Instant option
+        
+        /// The recurrence for this request
+        Recurrence : Recurrence
+        
+        /// The history entries for this request
+        History : History list
+        
+        /// The notes for this request
+        Notes : Note list
+    }
+
+/// Functions to support requests
+module Request =
+    
+    /// An empty request
+    let empty =
+        {   Id           = Cuid.generate () |> RequestId
+            EnteredOn    = Instant.MinValue
+            UserId       = UserId ""
+            SnoozedUntil = None
+            ShowAfter    = None
+            Recurrence   = Immediate
+            History      = []
+            Notes        = []
+        }
+
+
+/// JournalRequest is the form of a prayer request returned for the request journal display. It also contains
+/// properties that may be filled for history and notes.
+[<NoComparison; NoEquality>]
+type JournalRequest =
+    {   /// The ID of the request (just the CUID part)
+        RequestId : RequestId
+        
+        /// The ID of the user to whom the request belongs
+        UserId : UserId
+        
+        /// The current text of the request
+        Text : string
+        
+        /// The last time action was taken on the request
+        AsOf : Instant
+        
+        /// The last time a request was marked as prayed
+        LastPrayed : Instant option
+        
+        /// The last status for the request
+        LastStatus : RequestAction
+        
+        /// The time that this request should reappear in the user's journal
+        SnoozedUntil : Instant option
+        
+        /// The time after which this request should reappear in the user's journal by configured recurrence
+        ShowAfter : Instant option
+        
+        /// The recurrence for this request
+        Recurrence : Recurrence
+        
+        /// History entries for the request
+        History : History list
+        
+        /// Note entries for the request
+        Notes : Note list
+    }
+
+/// Functions to manipulate journal requests
+module JournalRequest =
+
+    /// Convert a request to the form used for the journal (precomputed values, no notes or history)
+    let ofRequestLite (req : Request) =
+        let history = Seq.ofList req.History
+        let lastHistory = Seq.tryHead history
+        // Requests are sorted by the "as of" field in this record; for sorting to work properly, we will put the
+        // largest of the last prayed date, the "snoozed until". or the "show after" date; if none of those are filled,
+        // we will use the last activity date. This will mean that:
+        //  - Immediately shown requests will be at the top of the list, in order from least recently prayed to most.
+        //  - Non-immediate requests will enter the list as if they were marked as prayed at that time; this will put
+        //    them at the bottom of the list.
+        //  - Snoozed requests will reappear at the bottom of the list when they return.
+        //  - New requests will go to the bottom of the list, but will rise as others are marked as prayed.
+        let lastActivity = lastHistory |> Option.map (_.AsOf) |> Option.defaultValue Instant.MinValue
+        let showAfter    = defaultArg req.ShowAfter    Instant.MinValue
+        let snoozedUntil = defaultArg req.SnoozedUntil Instant.MinValue
+        let lastPrayed   =
+            history
+            |> Seq.filter History.isPrayed
+            |> Seq.tryHead
+            |> Option.map (_.AsOf)
+            |> Option.defaultValue Instant.MinValue
+        let asOf = List.max [ lastPrayed; showAfter; snoozedUntil ]
+        {   RequestId    = req.Id
+            UserId       = req.UserId
+            Text         = history
+                           |> Seq.filter (fun it -> Option.isSome it.Text)
+                           |> Seq.tryHead
+                           |> Option.map (fun h -> Option.get h.Text)
+                           |> Option.defaultValue ""
+            AsOf         = if asOf > Instant.MinValue then asOf else lastActivity
+            LastPrayed   = if lastPrayed = Instant.MinValue then None else Some lastPrayed
+            LastStatus   = match lastHistory with Some h -> h.Status | None -> Created
+            SnoozedUntil = req.SnoozedUntil
+            ShowAfter    = req.ShowAfter
+            Recurrence   = req.Recurrence
+            History      = []
+            Notes        = []
+        }
+
+    /// Same as `ofRequestLite`, but with notes and history
+    let ofRequestFull req =
+        { ofRequestLite req with 
+            History = req.History
+            Notes   = req.Notes
+        }

src/MyPrayerJournal/Handlers.fs

@@ -1,24 +1,19 @@
-/// HTTP handlers for the myPrayerJournal API
+/// HTTP handlers for the myPrayerJournal API
 [<RequireQualifiedAccess>]
 module MyPrayerJournal.Handlers
 
-// fsharplint:disable RecordFieldNames
-
 open Giraffe
 open Giraffe.Htmx
-open Microsoft.AspNetCore.Authentication
-open Microsoft.AspNetCore.Http
 open System
-open System.Security.Claims
-open NodaTime
 
 /// Helper function to be able to split out log on
 [<AutoOpen>]
 module private LogOnHelpers =
 
+    open Microsoft.AspNetCore.Authentication
+    
     /// Log on, optionally specifying a redirected URL once authentication is complete
-  let logOn url : HttpHandler =
-    fun next ctx -> backgroundTask {
+    let logOn url : HttpHandler = fun next ctx -> task {
         match url with
         | Some it ->
             do! ctx.ChallengeAsync("Auth0", AuthenticationProperties(RedirectUri = it))
@@ -26,79 +21,91 @@ module private LogOnHelpers =
         | None -> return! challenge "Auth0" next ctx
     }
 
+
 /// Handlers for error conditions
 module Error =
 
     open Microsoft.Extensions.Logging
-  open System.Threading.Tasks
 
     /// Handle errors
     let error (ex : Exception) (log : ILogger) =
         log.LogError (EventId (), ex, "An unhandled exception has occurred while executing the request.")
         clearResponse
         >=> setStatusCode 500
-    >=> setHttpHeader "X-Toast" (sprintf "error|||%s: %s" (ex.GetType().Name) ex.Message)
+        >=> setHttpHeader "X-Toast" $"error|||{ex.GetType().Name}: {ex.Message}"
         >=> text ex.Message
 
-  /// Handle unauthorized actions, redirecting to log on for GETs, otherwise returning a 401 Not Authorized reponse
-  let notAuthorized : HttpHandler =
-    fun next ctx ->
-      (next, ctx)
-      ||> match ctx.Request.Method with
-          | "GET" -> logOn None
-          | _ -> setStatusCode 401 >=> fun _ _ -> Task.FromResult<HttpContext option> None
+    /// Handle unauthorized actions, redirecting to log on for GETs, otherwise returning a 401 Not Authorized response
+    let notAuthorized : HttpHandler = fun next ctx ->
+        (if ctx.Request.Method = "GET" then logOn None next else setStatusCode 401 earlyReturn) ctx
 
     /// Handle 404s from the API, sending known URL paths to the Vue app so that they can be handled there
     let notFound : HttpHandler =
         setStatusCode 404 >=> text "Not found"
 
 
+open System.Security.Claims
+open Microsoft.AspNetCore.Http
+open NodaTime
+
+/// Extensions on the HTTP context
+type HttpContext with
+    
+    /// The "sub" for the current user (None if no user is authenticated)
+    member this.CurrentUser =
+        this.User
+        |> Option.ofObj
+        |> Option.map (fun user -> user.Claims |> Seq.tryFind (fun u -> u.Type = ClaimTypes.NameIdentifier))
+        |> Option.flatten
+        |> Option.map (_.Value)
+    
+    /// The current user's ID
+    //  NOTE: this may raise if you don't run the request through the requireUser handler first
+    member this.UserId = UserId this.CurrentUser.Value
+    
+    /// The system clock
+    member this.Clock = this.GetService<IClock>()
+    
+    /// Get the current instant from the system clock
+    member this.Now = this.Clock.GetCurrentInstant
+    
+    /// Get the time zone from the X-Time-Zone header (default UTC)
+    member this.TimeZone =
+        match this.TryGetRequestHeader "X-Time-Zone" with
+        | Some tz ->
+            match this.GetService<IDateTimeZoneProvider>().GetZoneOrNull tz with
+            | null -> DateTimeZone.Utc
+            | zone -> zone
+        | None -> DateTimeZone.Utc
+
+
+open MyPrayerJournal.Data
+
 /// Handler helpers
 [<AutoOpen>]
 module private Helpers =
   
-  open LiteDB
     open Microsoft.Extensions.Logging
     open Microsoft.Net.Http.Headers
 
+    /// Require a user to be logged on
+    let requireUser : HttpHandler =
+        requiresAuthentication Error.notAuthorized
+    
+    /// Debug logger
     let debug (ctx : HttpContext) message =
         let fac = ctx.GetService<ILoggerFactory>()
         let log = fac.CreateLogger "Debug"
         log.LogInformation message
 
-  /// Get the LiteDB database
-  let db (ctx : HttpContext) = ctx.GetService<LiteDatabase>()
-
-  /// Get the user's "sub" claim
-  let user (ctx : HttpContext) =
-    ctx.User
-    |> Option.ofObj
-    |> Option.map (fun user -> user.Claims |> Seq.tryFind (fun u -> u.Type = ClaimTypes.NameIdentifier))
-    |> Option.flatten
-    |> Option.map (fun claim -> claim.Value)
-
-  /// Get the current user's ID
-  //  NOTE: this may raise if you don't run the request through the requiresAuthentication handler first
-  let userId ctx =
-    (user >> Option.get) ctx |> UserId
-
-  /// Get the system clock
-  let clock (ctx : HttpContext) =
-    ctx.GetService<IClock> ()
-  
-  /// Get the current instant
-  let now ctx =
-    (clock ctx).GetCurrentInstant ()
-
     /// Return a 201 CREATED response
     let created =
         setStatusCode 201
 
     /// Return a 201 CREATED response with the location header set for the created resource
-  let createdAt url : HttpHandler =
-    fun next ctx ->
-      (sprintf "%s://%s%s" ctx.Request.Scheme ctx.Request.Host.Value url |> setHttpHeader HeaderNames.Location
-       >=> created) next ctx
+    let createdAt url : HttpHandler = fun next ctx ->
+        Successful.CREATED
+            ($"{ctx.Request.Scheme}://{ctx.Request.Host.Value}{url}" |> setHttpHeader HeaderNames.Location) next ctx
   
     /// Return a 303 SEE OTHER response (forces a GET on the redirected URL)
     let seeOther (url : string) =
@@ -107,50 +114,50 @@ module private Helpers =
     /// Render a component result
     let renderComponent nodes : HttpHandler =
         noResponseCaching
-    >=> fun next ctx -> backgroundTask {
+        >=> fun _ ctx -> backgroundTask {
             return! ctx.WriteHtmlStringAsync(ViewEngine.RenderView.AsString.htmlNodes nodes)
         }
 
     open Views.Layout
+    open System.Threading.Tasks
     
     /// Create a page rendering context
     let pageContext (ctx : HttpContext) pageTitle content = backgroundTask {
-    let! hasSnoozed = backgroundTask {
-      match user ctx with
-      | Some _ -> return! Data.hasSnoozed (userId ctx) (now ctx) (db ctx)
-      | None   -> return  false
-      }
-    return {
-      isAuthenticated = (user >> Option.isSome) ctx
-      hasSnoozed      = hasSnoozed
-      currentUrl      = ctx.Request.Path.Value
-      pageTitle       = pageTitle
-      content         = content
+        let! hasSnoozed =
+            match ctx.CurrentUser with
+            | Some _ -> Journal.hasSnoozed ctx.UserId (ctx.Now())
+            | None   -> Task.FromResult false
+        return
+            {   IsAuthenticated = Option.isSome ctx.CurrentUser
+                HasSnoozed      = hasSnoozed
+                CurrentUrl      = ctx.Request.Path.Value
+                PageTitle       = pageTitle
+                Content         = content
             }
     }
 
     /// Composable handler to write a view to the output
-  let writeView view : HttpHandler =
-    fun next ctx -> backgroundTask {
+    let writeView view : HttpHandler = fun _ ctx -> backgroundTask {
         return! ctx.WriteHtmlViewAsync view
     }
     
+    
     /// Hold messages across redirects
     module Messages =
 
         /// The messages being held
-    let mutable private messages : Map<string, (string * string)> = Map.empty
+        let mutable private messages : Map<UserId, string * string> = Map.empty
 
         /// Locked update to prevent updates by multiple threads
         let private upd8 = obj ()
 
         /// Push a new message into the list
-    let push ctx message url = lock upd8 (fun () ->
-      messages <- messages.Add (ctx |> (user >> Option.get), (message, url)))
+        let push (ctx : HttpContext) message url = lock upd8 (fun () ->
+            messages <- messages.Add(ctx.UserId, (message, url)))
 
         /// Add a success message header to the response
         let pushSuccess ctx message url =
-      push ctx (sprintf "success|||%s" message) url
+            push ctx $"success|||%s{message}" url
         
         /// Pop the messages for the given user
         let pop userId = lock upd8 (fun () ->
@@ -159,17 +166,16 @@ module private Helpers =
             msg)
 
     /// Send a partial result if this is not a full page load (does not append no-cache headers)
-  let partialStatic (pageTitle : string) content : HttpHandler =
-    fun next ctx -> backgroundTask {
+    let partialStatic (pageTitle : string) content : HttpHandler = fun next ctx -> task {
         let  isPartial = ctx.Request.IsHtmx && not ctx.Request.IsHtmxRefresh
         let! pageCtx   = pageContext ctx pageTitle content
         let  view      = (match isPartial with true -> partial | false -> view) pageCtx
         return! 
             (next, ctx)
-        ||> match user ctx with
-            | Some u ->
-                match Messages.pop u with
-                | Some (msg, url) -> setHttpHeader "X-Toast" msg >=> withHxPush url >=> writeView view
+            ||> match ctx.CurrentUser with
+                | Some _ ->
+                    match Messages.pop ctx.UserId with
+                    | Some (msg, url) -> setHttpHeader "X-Toast" msg >=> withHxPushUrl url >=> writeView view
                     | None -> writeView view
                 | None -> writeView view
     }
@@ -192,104 +198,109 @@ module Models =
   
     /// An additional note
     [<CLIMutable; NoComparison; NoEquality>]
-  type NoteEntry = {
-    /// The notes being added
+    type NoteEntry =
+        {   /// The notes being added
             notes : string
         }
     
     /// A prayer request
     [<CLIMutable; NoComparison; NoEquality>]
-  type Request = {
-    /// The ID of the request
+    type Request =
+        {   /// The ID of the request
             requestId     : string
+            
             /// Where to redirect after saving
             returnTo      : string
+            
             /// The text of the request
             requestText   : string
+            
             /// The additional status to record
             status        : string option
+            
             /// The recurrence type
             recurType     : string
+            
             /// The recurrence count
             recurCount    : int16 option
+            
             /// The recurrence interval
             recurInterval : string option
         }
   
     /// The date until which a request should not appear in the journal
     [<CLIMutable; NoComparison; NoEquality>]
-  type SnoozeUntil = {
-    /// The date (YYYY-MM-DD) at which the request should reappear
+    type SnoozeUntil =
+        {   /// The date (YYYY-MM-DD) at which the request should reappear
             until : string
         }
 
 
-open MyPrayerJournal.Data.Extensions
 open NodaTime.Text
 
 /// Handlers for less-than-full-page HTML requests
 module Components =
 
     // GET /components/journal-items
-  let journalItems : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let  now   = now ctx
-      let! jrnl  = Data.journalByUserId (userId ctx) (db ctx)
-      let  shown = jrnl |> List.filter (fun it -> now > it.snoozedUntil && now > it.showAfter)
-      return! renderComponent [ Views.Journal.journalItems now shown ] next ctx
+    let journalItems : HttpHandler = requireUser >=> fun next ctx -> task {
+        let now = ctx.Now ()
+        let shouldBeShown (req : JournalRequest) =
+            match req.SnoozedUntil, req.ShowAfter with
+            | None, None -> true
+            | Some snooze, Some hide when snooze < now && hide < now -> true
+            | Some snooze, _ when snooze < now -> true
+            | _, Some hide when hide < now -> true
+            | _, _ -> false
+        let! journal = Journal.forUser ctx.UserId
+        let  shown   = journal |> List.filter shouldBeShown
+        return! renderComponent [ Views.Journal.journalItems now ctx.TimeZone shown ] next ctx
     }
   
     // GET /components/request-item/[req-id]
-  let requestItem reqId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      match! Data.tryJournalById (RequestId.ofString reqId) (userId ctx) (db ctx) with
-      | Some req -> return! renderComponent [ Views.Request.reqListItem (now ctx) req ] next ctx
+    let requestItem reqId : HttpHandler = requireUser >=> fun next ctx -> task {
+        match! Journal.tryById (RequestId.ofString reqId) ctx.UserId with
+        | Some req -> return! renderComponent [ Views.Request.reqListItem (ctx.Now()) ctx.TimeZone req ] next ctx
         | None     -> return! Error.notFound next ctx
     }
 
     // GET /components/request/[req-id]/add-notes
     let addNotes requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> renderComponent (Views.Journal.notesEdit (RequestId.ofString requestId))
+        requireUser >=> renderComponent (Views.Journal.notesEdit (RequestId.ofString requestId))
 
     // GET /components/request/[req-id]/notes
-  let notes requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let! notes = Data.notesById (RequestId.ofString requestId) (userId ctx) (db ctx)
-      return! renderComponent (Views.Request.notes (now ctx) notes) next ctx
+    let notes requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let! notes = Note.byRequestId (RequestId.ofString requestId) ctx.UserId
+        return! renderComponent (Views.Request.notes (ctx.Now()) ctx.TimeZone notes) next ctx
     }
   
     // GET /components/request/[req-id]/snooze
     let snooze requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> renderComponent [ RequestId.ofString requestId |> Views.Journal.snooze ]
+        requireUser >=> renderComponent [ RequestId.ofString requestId |> Views.Journal.snooze ]
 
 
-/// / URL    
+/// / URL and documentation
 module Home =
   
     // GET /
     let home : HttpHandler =
         partialStatic "Welcome!" Views.Layout.home
   
+    // GET /docs
+    let docs : HttpHandler =
+        partialStatic "Documentation" Views.Docs.index
 
 /// /journal URL
 module Journal =
   
     // GET /journal
-  let journal : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
+    let journal : HttpHandler = requireUser >=> fun next ctx -> task {
         let usr =
             ctx.User.Claims
             |> Seq.tryFind (fun c -> c.Type = ClaimTypes.GivenName)
-        |> Option.map (fun c -> c.Value)
+            |> Option.map (_.Value)
             |> Option.defaultValue "Your"
-      let title = usr |> match usr with "Your" -> sprintf "%s" | _ -> sprintf "%s's"
-      return! partial (sprintf "%s Prayer Journal" title) (Views.Journal.journal usr) next ctx
+        let title = usr |> match usr with "Your" -> sprintf "%s" | _ -> sprintf "%s&rsquo;s"
+        return! partial $"{title} Prayer Journal" (Views.Journal.journal usr) next ctx
     }
 
 
@@ -308,12 +319,12 @@ module Legal =
 /// /api/request and /request(s) URLs
 module Request =
     
+    open Cuid
+    
     // GET /request/[req-id]/edit  
-  let edit requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
+    let edit requestId : HttpHandler = requireUser >=> fun next ctx -> task {
         let returnTo =
-        match ctx.Request.Headers.Referer.[0] with
+            match ctx.Request.Headers.Referer[0] with
             | it when it.EndsWith "/active"  -> "active"
             | it when it.EndsWith "/snoozed" -> "snoozed"
             | _                              -> "journal"
@@ -322,7 +333,7 @@ module Request =
             return! partial "Add Prayer Request"
                         (Views.Request.edit (JournalRequest.ofRequestLite Request.empty) returnTo true) next ctx
         | _     ->
-          match! Data.tryJournalById (RequestId.ofString requestId) (userId ctx) (db ctx) with
+            match! Journal.tryById (RequestId.ofString requestId) ctx.UserId with
             | Some req ->
                 debug ctx "Found - sending view"
                 return! partial "Edit Prayer Request" (Views.Request.edit req returnTo false) next ctx
@@ -332,197 +343,153 @@ module Request =
     }
 
     // PATCH /request/[req-id]/prayed
-  let prayed requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let db    = db     ctx
-      let usrId = userId ctx
+    let prayed requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let userId = ctx.UserId
         let reqId  = RequestId.ofString requestId
-      match! Data.tryRequestById reqId usrId db with
+        match! Journal.tryById reqId userId with
         | Some req ->
-          let now  = now ctx
-          do! Data.addHistory reqId usrId { asOf = now; status = Prayed; text = None } db
+            let now  = ctx.Now ()
+            do! History.add reqId userId { AsOf = now; Status = Prayed; Text = None }
             let nextShow =
-            match Recurrence.duration req.recurType with
-            | 0L       -> Instant.MinValue
-            | duration -> now.Plus (Duration.FromSeconds (duration * int64 req.recurCount))
-          do! Data.updateShowAfter reqId usrId nextShow db
-          do! db.saveChanges ()
+                match Recurrence.duration req.Recurrence with
+                | 0L       -> None
+                | duration -> Some <| now.Plus (Duration.FromSeconds duration)
+            do! Request.updateShowAfter reqId userId nextShow
             return! (withSuccessMessage "Request marked as prayed" >=> Components.journalItems) next ctx
         | None -> return! Error.notFound next ctx
     }
   
-  /// POST /request/[req-id]/note
-  let addNote requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let db    = db     ctx
-      let usrId = userId ctx
+    // POST /request/[req-id]/note
+    let addNote requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let userId = ctx.UserId
         let reqId  = RequestId.ofString requestId
-      match! Data.tryRequestById reqId usrId db with
-      | Some _ ->
+        match! Request.existsById reqId userId with
+        | true ->
             let! notes = ctx.BindFormAsync<Models.NoteEntry>()
-          do! Data.addNote reqId usrId { asOf = now ctx; notes = notes.notes } db
-          do! db.saveChanges ()
+            do! Note.add reqId userId { AsOf = ctx.Now(); Notes = notes.notes }
             return! (withSuccessMessage "Added Notes" >=> hideModal "notes" >=> created) next ctx
-      | None -> return! Error.notFound next ctx
+        | false -> return! Error.notFound next ctx
     }
           
     // GET /requests/active
-  let active : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let! reqs = Data.journalByUserId (userId ctx) (db ctx)
-      return! partial "Active Requests" (Views.Request.active (now ctx) reqs) next ctx
+    let active : HttpHandler = requireUser >=> fun next ctx -> task {
+        let! reqs = Journal.forUser ctx.UserId
+        return! partial "Active Requests" (Views.Request.active (ctx.Now()) ctx.TimeZone reqs) next ctx
     }
   
     // GET /requests/snoozed
-  let snoozed : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let! reqs    = Data.journalByUserId (userId ctx) (db ctx)
-      let  now     = now ctx
-      let  snoozed = reqs |> List.filter (fun it -> it.snoozedUntil > now)
-      return! partial "Active Requests" (Views.Request.snoozed now snoozed) next ctx
+    let snoozed : HttpHandler = requireUser >=> fun next ctx -> task {
+        let! reqs    = Journal.forUser ctx.UserId
+        let  now     = ctx.Now()
+        let  snoozed = reqs
+                       |> List.filter (fun it -> defaultArg (it.SnoozedUntil |> Option.map (fun it -> it > now)) false)
+        return! partial "Snoozed Requests" (Views.Request.snoozed now ctx.TimeZone snoozed) next ctx
     }
 
     // GET /requests/answered
-  let answered : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let! reqs = Data.answeredRequests (userId ctx) (db ctx)
-      return! partial "Answered Requests" (Views.Request.answered (now ctx) reqs) next ctx
-      }
-  
-  // GET /api/request/[req-id]
-  let get requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      match! Data.tryJournalById (RequestId.ofString requestId) (userId ctx) (db ctx) with
-      | Some req -> return! json req next ctx
-      | None -> return! Error.notFound next ctx
+    let answered : HttpHandler = requireUser >=> fun next ctx -> task {
+        let! reqs = Journal.answered ctx.UserId
+        return! partial "Answered Requests" (Views.Request.answered (ctx.Now()) ctx.TimeZone reqs) next ctx
     }
   
     // GET /request/[req-id]/full
-  let getFull requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      match! Data.tryFullRequestById (RequestId.ofString requestId) (userId ctx) (db ctx) with
-      | Some req -> return! partial "Prayer Request" (Views.Request.full (clock ctx) req) next ctx
+    let getFull requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        match! Request.tryById (RequestId.ofString requestId) ctx.UserId with
+        | Some req -> return! partial "Prayer Request" (Views.Request.full ctx.Clock ctx.TimeZone req) next ctx
         | None     -> return! Error.notFound next ctx
     }
   
     // PATCH /request/[req-id]/show
-  let show requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let db    = db     ctx
-      let usrId = userId ctx
+    let show requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let userId = ctx.UserId
         let reqId  = RequestId.ofString requestId
-      match! Data.tryRequestById reqId usrId db with
-      | Some _ ->
-          do! Data.updateShowAfter reqId usrId Instant.MinValue db
-          do! db.saveChanges ()
+        match! Request.existsById reqId userId with
+        | true ->
+            do! Request.updateShowAfter reqId userId None
             return! (withSuccessMessage "Request now shown" >=> Components.requestItem requestId) next ctx
-      | None -> return! Error.notFound next ctx
+        | false -> return! Error.notFound next ctx
     }
   
     // PATCH /request/[req-id]/snooze
-  let snooze requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let db    = db     ctx
-      let usrId = userId ctx
+    let snooze requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let userId = ctx.UserId
         let reqId  = RequestId.ofString requestId
-      match! Data.tryRequestById reqId usrId db with
-      | Some _ ->
+        match! Request.existsById reqId userId with
+        | true ->
             let! until = ctx.BindFormAsync<Models.SnoozeUntil>()
             let date =
                 LocalDatePattern.CreateWithInvariantCulture("yyyy-MM-dd").Parse(until.until).Value
                     .AtStartOfDayInZone(DateTimeZone.Utc)
                     .ToInstant()
-          do! Data.updateSnoozed reqId usrId date db
-          do! db.saveChanges ()
+            do! Request.updateSnoozed reqId userId (Some date)
             return!
                 (withSuccessMessage $"Request snoozed until {until.until}"
                  >=> hideModal "snooze"
                  >=> Components.journalItems) next ctx
-      | None -> return! Error.notFound next ctx
+        | false -> return! Error.notFound next ctx
     }
   
     // PATCH /request/[req-id]/cancel-snooze
-  let cancelSnooze requestId : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
-      let db    = db     ctx
-      let usrId = userId ctx
+    let cancelSnooze requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+        let userId = ctx.UserId
         let reqId  = RequestId.ofString requestId
-      match! Data.tryRequestById reqId usrId db with
-      | Some _ ->
-          do! Data.updateSnoozed reqId usrId Instant.MinValue db
-          do! db.saveChanges ()
+        match! Request.existsById reqId userId with
+        | true ->
+            do! Request.updateSnoozed reqId userId None
             return! (withSuccessMessage "Request unsnoozed" >=> Components.requestItem requestId) next ctx
-      | None -> return! Error.notFound next ctx
+        | false -> return! Error.notFound next ctx
     }
 
-  /// Derive a recurrence and interval from its primitive representation in the form
+    /// Derive a recurrence from its representation in the form
     let private parseRecurrence (form : Models.Request) =
-    (Recurrence.ofString (match form.recurInterval with Some x -> x | _ -> "Immediate"),
-     defaultArg form.recurCount (int16 0))
+        match form.recurInterval with Some x -> $"{defaultArg form.recurCount 0s} {x}" | None -> "Immediate"
+        |> Recurrence.ofString
 
     // POST /request
-  let add : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
+    let add : HttpHandler = requireUser >=> fun next ctx -> task {
         let! form   = ctx.BindModelAsync<Models.Request>()
-      let  db               = db ctx
-      let  usrId            = userId ctx
-      let  now              = now ctx
-      let (recur, interval) = parseRecurrence form
+        let  userId = ctx.UserId
+        let  now    = ctx.Now()
         let  req    =
             { Request.empty with
-            userId     = usrId
-            enteredOn  = now
-            showAfter  = Instant.MinValue
-            recurType  = recur
-            recurCount = interval
-            history    = [
-              { asOf   = now
-                status = Created
-                text   = Some form.requestText
+                Id         = Cuid.generate () |> RequestId
+                UserId     = userId
+                EnteredOn  = now
+                ShowAfter  = None
+                Recurrence = parseRecurrence form
+                History    = [
+                    {   AsOf   = now
+                        Status = Created
+                        Text   = Some form.requestText
                     }      
                 ]
             }
-      Data.addRequest req db
-      do! db.saveChanges ()
+        do! Request.add req
         Messages.pushSuccess ctx "Added prayer request" "/journal"
         return! seeOther "/journal" next ctx
     }
   
     // PATCH /request
-  let update : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> backgroundTask {
+    let update : HttpHandler = requireUser >=> fun next ctx -> task {
         let! form   = ctx.BindModelAsync<Models.Request>()
-      let  db    = db ctx
-      let  usrId = userId ctx
-      match! Data.tryJournalById (RequestId.ofString form.requestId) usrId db with
+        let  userId = ctx.UserId
+        // TODO: update the instance and save rather than all these little updates
+        match! Journal.tryById (RequestId.ofString form.requestId) userId with
         | Some req ->
             // update recurrence if changed
-          let (recur, interval) = parseRecurrence form
-          match recur = req.recurType && interval = req.recurCount with
+            let recur = parseRecurrence form
+            match recur = req.Recurrence with
             | true  -> ()
             | false ->
-              do! Data.updateRecurrence req.requestId usrId recur interval db
+                do! Request.updateRecurrence req.RequestId userId recur
                 match recur with
-              | Immediate -> do! Data.updateShowAfter req.requestId usrId Instant.MinValue db
+                | Immediate -> do! Request.updateShowAfter req.RequestId userId None
                 | _         -> ()
             // append history
             let upd8Text = form.requestText.Trim()
-          let text     = match upd8Text = req.text with true -> None | false -> Some upd8Text
-          do! Data.addHistory req.requestId usrId
-                { asOf = now ctx; status = (Option.get >> RequestAction.ofString) form.status; text = text } db
-          do! db.saveChanges ()
+            let text     = if upd8Text = req.Text then None else Some upd8Text
+            do! History.add req.RequestId userId
+                    { AsOf = ctx.Now(); Status = (Option.get >> RequestAction.ofString) form.status; Text = text }
             let nextUrl =
                 match form.returnTo with
                 | "active"          -> "/requests/active"
@@ -537,6 +504,7 @@ module Request =
 /// Handlers for /user URLs
 module User =
 
+    open Microsoft.AspNetCore.Authentication
     open Microsoft.AspNetCore.Authentication.Cookies
 
     // GET /user/log-on
@@ -544,9 +512,7 @@ module User =
         logOn (Some "/journal")
   
     // GET /user/log-off
-  let logOff : HttpHandler =
-    requiresAuthentication Error.notAuthorized
-    >=> fun next ctx -> task {
+    let logOff : HttpHandler = requireUser >=> fun next ctx -> task {
         do! ctx.SignOutAsync("Auth0", AuthenticationProperties (RedirectUri = "/"))
         do! ctx.SignOutAsync CookieAuthenticationDefaults.AuthenticationScheme
         return! next ctx
@@ -556,8 +522,8 @@ module User =
 open Giraffe.EndpointRouting
 
 /// The routes for myPrayerJournal
-let routes =
-  [ GET_HEAD [ route "/" Home.home ]
+let routes = [
+    GET_HEAD [ route "/" Home.home ]
     subRoute "/components/" [
         GET_HEAD [
             route  "journal-items"        Components.journalItems
@@ -567,6 +533,7 @@ let routes =
             routef "request/%s/snooze"    Components.snooze
         ]
     ]
+    GET_HEAD [ route "/docs"    Home.docs ]
     GET_HEAD [ route "/journal" Journal.journal ]
     subRoute "/legal/" [
         GET_HEAD [

src/MyPrayerJournal/MyPrayerJournal.fsproj

@@ -1,7 +1,11 @@
-<Project Sdk="Microsoft.NET.Sdk.Web">
+<Project Sdk="Microsoft.NET.Sdk.Web">
   <PropertyGroup>
-    <TargetFramework>net6.0</TargetFramework>
-    <Version>3.0.0.0</Version>
+    <TargetFramework>net8.0</TargetFramework>
+    <Version>3.4</Version>
+    <DebugType>embedded</DebugType>
+    <GenerateDocumentationFile>false</GenerateDocumentationFile>
+    <PublishSingleFile>false</PublishSingleFile>
+    <SelfContained>false</SelfContained>
   </PropertyGroup>
   <ItemGroup>
     <Compile Include="Domain.fs" />
@@ -12,20 +16,21 @@
     <Compile Include="Views/Layout.fs" />
     <Compile Include="Views/Legal.fs" />
     <Compile Include="Views/Request.fs" />
+    <Compile Include="Views\Docs.fs" />
     <Compile Include="Handlers.fs" />
     <Compile Include="Program.fs" />
   </ItemGroup>
   <ItemGroup>
-    <PackageReference Include="FSharp.SystemTextJson" Version="0.17.4" />
+    <PackageReference Include="BitBadger.Documents.Postgres" Version="3.1.0" />
+    <PackageReference Include="FSharp.SystemTextJson" Version="1.3.13" />
     <PackageReference Include="FunctionalCuid" Version="1.0.0" />
-    <PackageReference Include="Giraffe" Version="5.0.0" />
-    <PackageReference Include="LiteDB" Version="5.0.11" />
-    <PackageReference Include="Microsoft.AspNetCore.Authentication.OpenIdConnect" Version="5.0.10" />
-    <PackageReference Include="NodaTime" Version="3.0.9" />
-  </ItemGroup>
-  <ItemGroup>
-    <ProjectReference Include="../../../Giraffe.Htmx/src/Htmx/Giraffe.Htmx.fsproj" />
-    <ProjectReference Include="../../../Giraffe.Htmx/src/ViewEngine.Htmx/Giraffe.ViewEngine.Htmx.fsproj" />
+    <PackageReference Include="Giraffe" Version="6.4.0" />
+    <PackageReference Include="Giraffe.Htmx" Version="1.9.12" />
+    <PackageReference Include="Giraffe.ViewEngine.Htmx" Version="1.9.12" />
+    <PackageReference Include="Microsoft.AspNetCore.Authentication.OpenIdConnect" Version="8.0.6" />
+    <PackageReference Include="NodaTime.Serialization.SystemTextJson" Version="1.2.0" />
+    <PackageReference Include="Npgsql.NodaTime" Version="8.0.3" />
+    <PackageReference Update="FSharp.Core" Version="8.0.300" />
   </ItemGroup>
   <ItemGroup>
     <Folder Include="wwwroot\" />

src/MyPrayerJournal/Program.fs

@@ -1,102 +1,62 @@
 module MyPrayerJournal.Api
 
-open Microsoft.AspNetCore.Builder
-open Microsoft.AspNetCore.Hosting
-open System.IO
-
-/// Configuration functions for the application
-module Configure =
-  
-  /// Configure the content root
-  let contentRoot root =
-    WebApplicationOptions (ContentRootPath = root) |> WebApplication.CreateBuilder
-
-
-  open Microsoft.Extensions.Configuration
-
-  /// Configure the application configuration
-  let appConfiguration (bldr : WebApplicationBuilder) =
-    bldr.Configuration
-      .SetBasePath(bldr.Environment.ContentRootPath)
-      .AddJsonFile("appsettings.json", optional = false, reloadOnChange = true)
-      .AddJsonFile($"appsettings.{bldr.Environment.EnvironmentName}.json", optional = true, reloadOnChange = true)
-      .AddEnvironmentVariables ()
-    |> ignore
-    bldr
-
-
-  open Microsoft.AspNetCore.Server.Kestrel.Core
-
-  /// Configure Kestrel from appsettings.json
-  let kestrel (bldr : WebApplicationBuilder) =
-    let kestrelOpts (ctx : WebHostBuilderContext) (opts : KestrelServerOptions) =
-      (ctx.Configuration.GetSection >> opts.Configure >> ignore) "Kestrel"
-    bldr.WebHost.UseKestrel().ConfigureKestrel kestrelOpts |> ignore
-    bldr
-
-
-  /// Configure the web root directory
-  let webRoot pathSegments (bldr : WebApplicationBuilder) =
-    Array.concat [ [| bldr.Environment.ContentRootPath |]; pathSegments ]
-    |> (Path.Combine >> bldr.WebHost.UseWebRoot >> ignore)
-    bldr
-
-
-  open Microsoft.Extensions.Logging
-  open Microsoft.Extensions.Hosting
-
-  /// Configure logging
-  let logging (bldr : WebApplicationBuilder) =
-    match bldr.Environment.IsDevelopment () with
-    | true -> ()
-    | false -> bldr.Logging.AddFilter (fun l -> l > LogLevel.Information) |> ignore
-    bldr.Logging.AddConsole().AddDebug() |> ignore
-    bldr
-
-
-  open Giraffe
-  open LiteDB
-  open Microsoft.AspNetCore.Authentication.Cookies
-  open Microsoft.AspNetCore.Authentication.OpenIdConnect
 open Microsoft.AspNetCore.Http
-  open Microsoft.Extensions.DependencyInjection
-  open Microsoft.IdentityModel.Protocols.OpenIdConnect
-  open NodaTime
-  open System
-  open System.Text.Json
-  open System.Text.Json.Serialization
-  open System.Threading.Tasks
 
-  /// Configure dependency injection
-  let services (bldr : WebApplicationBuilder) =
 let sameSite (opts : CookieOptions) =
     match opts.SameSite, opts.Secure with
     | SameSiteMode.None, false -> opts.SameSite <- SameSiteMode.Unspecified
     | _, _ -> ()
 
-    bldr.Services
-      .AddRouting()
-      .AddGiraffe()
-      .AddSingleton<IClock>(SystemClock.Instance)
-      .Configure<CookiePolicyOptions>(
-        fun (opts : CookiePolicyOptions) ->
+open Giraffe
+open Giraffe.EndpointRouting
+open Microsoft.AspNetCore.Authentication.Cookies
+open Microsoft.AspNetCore.Authentication.OpenIdConnect
+open Microsoft.AspNetCore.Builder
+open Microsoft.AspNetCore.HttpOverrides
+open Microsoft.Extensions.Configuration
+open Microsoft.Extensions.DependencyInjection
+open Microsoft.Extensions.Hosting
+open Microsoft.Extensions.Logging
+open Microsoft.IdentityModel.Protocols.OpenIdConnect
+open MyPrayerJournal.Data
+open NodaTime
+open System
+open System.Text.Json
+open System.Threading.Tasks
+
+[<EntryPoint>]
+let main args =
+    //use host = Configure.webHost [| "wwwroot" |] (Directory.GetCurrentDirectory ())
+    //host.Run ()
+    let builder = WebApplication.CreateBuilder args
+    let _       = builder.Configuration.AddEnvironmentVariables "MPJ_"
+    let svc     = builder.Services
+    let cfg     = svc.BuildServiceProvider().GetRequiredService<IConfiguration>()
+
+    let _ = svc.AddRouting()
+    let _ = svc.AddGiraffe()
+    let _ = svc.AddSingleton<IClock> SystemClock.Instance
+    let _ = svc.AddSingleton<IDateTimeZoneProvider> DateTimeZoneProviders.Tzdb
+    let _ = svc.Configure<ForwardedHeadersOptions>(fun (opts : ForwardedHeadersOptions) ->
+                opts.ForwardedHeaders <- ForwardedHeaders.XForwardedFor ||| ForwardedHeaders.XForwardedProto)
+    
+    let _ =
+        svc.Configure<CookiePolicyOptions>(fun (opts : CookiePolicyOptions) ->
             opts.MinimumSameSitePolicy <- SameSiteMode.Unspecified
             opts.OnAppendCookie        <- fun ctx -> sameSite ctx.CookieOptions
             opts.OnDeleteCookie        <- fun ctx -> sameSite ctx.CookieOptions)
-      .AddAuthentication(
-        /// Use HTTP "Bearer" authentication with JWTs
-        fun opts ->
+    let _ =
+        svc.AddAuthentication(fun opts ->
             opts.DefaultAuthenticateScheme <- CookieAuthenticationDefaults.AuthenticationScheme
             opts.DefaultSignInScheme       <- CookieAuthenticationDefaults.AuthenticationScheme
             opts.DefaultChallengeScheme    <- CookieAuthenticationDefaults.AuthenticationScheme)
             .AddCookie()
-      .AddOpenIdConnect("Auth0",
-        /// Configure OIDC with Auth0 options from configuration
-        fun opts ->
-          let cfg = bldr.Configuration.GetSection "Auth0"
-          opts.Authority    <- sprintf "https://%s/" cfg["Domain"]
-          opts.ClientId     <- cfg["Id"]
-          opts.ClientSecret <- cfg["Secret"]
+            .AddOpenIdConnect("Auth0", fun opts ->
+                // Configure OIDC with Auth0 options from configuration
+                let auth0 = cfg.GetSection "Auth0"
+                opts.Authority    <- $"""https://{auth0["Domain"]}/"""
+                opts.ClientId     <- auth0["Id"]
+                opts.ClientSecret <- auth0["Secret"]
                 opts.ResponseType <- OpenIdConnectResponseType.Code
                 
                 opts.Scope.Clear()
@@ -118,64 +78,34 @@ module Configure =
                                 | true ->
                                     // transform to absolute
                                     let request = ctx.Request
-                        sprintf "%s://%s%s%s" request.Scheme request.Host.Value request.PathBase.Value redirUri
+                                    $"{request.Scheme}://{request.Host.Value}{request.PathBase.Value}{redirUri}"
                                 | false -> redirUri
-                  Uri.EscapeDataString finalRedirUri |> sprintf "&returnTo=%s"
-            sprintf "https://%s/v2/logout?client_id=%s%s" cfg["Domain"] cfg["Id"] returnTo
-            |> ctx.Response.Redirect
+                            Uri.EscapeDataString $"&returnTo={finalRedirUri}"
+                    ctx.Response.Redirect $"""https://{auth0["Domain"]}/v2/logout?client_id={auth0["Id"]}{returnTo}"""
                     ctx.HandleResponse()
-
                     Task.CompletedTask
                 opts.Events.OnRedirectToIdentityProvider <- fun ctx ->
-            let bldr = UriBuilder ctx.ProtocolMessage.RedirectUri
-            bldr.Scheme <- cfg["Scheme"]
-            bldr.Port   <- int cfg["Port"]
-            ctx.ProtocolMessage.RedirectUri <- string bldr
-            Task.CompletedTask
-          )
-    |> ignore
-    let jsonOptions = JsonSerializerOptions ()
-    jsonOptions.Converters.Add (JsonFSharpConverter ())
-    let db = new LiteDatabase (bldr.Configuration.GetConnectionString "db")
-    Data.Startup.ensureDb db
-    bldr.Services.AddSingleton(jsonOptions)
-      .AddSingleton<Json.ISerializer, SystemTextJson.Serializer>()
-      .AddSingleton<LiteDatabase> db
-    |> ignore
-    bldr.Build ()
+                    let uri = UriBuilder ctx.ProtocolMessage.RedirectUri
+                    uri.Scheme <- auth0["Scheme"]
+                    uri.Port   <- int auth0["Port"]
+                    ctx.ProtocolMessage.RedirectUri <- string uri
+                    Task.CompletedTask)
     
+    let _ = svc.AddSingleton<JsonSerializerOptions> Json.options
+    let _ = svc.AddSingleton<Json.ISerializer>(SystemTextJson.Serializer Json.options)
+    let _ = Connection.setUp cfg |> Async.AwaitTask |> Async.RunSynchronously
     
-  open Giraffe.EndpointRouting
+    if builder.Environment.IsDevelopment() then builder.Logging.AddFilter(fun l -> l > LogLevel.Information) |> ignore
+    let _ = builder.Logging.AddConsole().AddDebug() |> ignore
 
-  /// Configure the web application
-  let application (app : WebApplication) =
-    // match app.Environment.IsDevelopment () with
-    // | true -> app.UseDeveloperExceptionPage ()
-    // | false -> app.UseGiraffeErrorHandler Handlers.Error.error
-    // |> ignore
-    app
-      .UseStaticFiles()
-      .UseCookiePolicy()
-      .UseRouting()
-      .UseAuthentication()
-      .UseGiraffeErrorHandler(Handlers.Error.error)
-      .UseEndpoints (fun e -> e.MapGiraffeEndpoints Handlers.routes |> ignore)
-    |> ignore
-    app
+    use app = builder.Build()
+    let _ = app.UseStaticFiles()
+    let _ = app.UseCookiePolicy()
+    let _ = app.UseRouting()
+    let _ = app.UseAuthentication()
+    let _ = app.UseGiraffeErrorHandler Handlers.Error.error
+    let _ = app.UseEndpoints(fun e -> e.MapGiraffeEndpoints Handlers.routes)
     
-  /// Compose all the configurations into one
-  let webHost pathSegments =
-    contentRoot
-    >> appConfiguration
-    >> kestrel
-    >> webRoot pathSegments
-    >> logging
-    >> services
-    >> application
+    app.Run()
 
-
-[<EntryPoint>]
-let main _ =
-  use host = Configure.webHost [| "wwwroot" |] (Directory.GetCurrentDirectory ())
-  host.Run ()
     0

src/MyPrayerJournal/Views/Docs.fs

@@ -0,0 +1,184 @@
+module MyPrayerJournal.Views.Docs
+
+open Giraffe.ViewEngine
+
+/// The "About myPrayerJournal" section
+let private about = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "About myPrayerJournal" ]
+    p [] [
+        rawText "Journaling has a long history; it helps people remember what happened, and the act of writing helps "
+        rawText "people think about what happened and process it. A prayer journal is not a new concept; it helps you "
+        rawText "keep track of the requests for which you've prayed, you can use it to pray over things repeatedly, "
+        rawText "and you can write the result when the answer comes "; em [] [ rawText "(or it was “no”)" ]
+        rawText "."
+    ]
+    p [] [
+        rawText "myPrayerJournal was borne of out of a personal desire "
+        a [ _href "https://daniel.summershome.org"; _target "_blank"; _rel "noopener" ] [ rawText "Daniel" ]
+        rawText " had to have something that would help him with his prayer life. When it’s time to pray, "
+        rawText "it’s not really time to use an app, so the design goal here is to keep it simple and "
+        rawText "unobtrusive. It will also help eliminate some of the downsides to a paper prayer journal, like not "
+        rawText "remembering whether you’ve prayed for a request, or running out of room to write another update "
+        rawText "on one."
+    ]
+]
+
+/// The "Signing Up" section
+let private signUp = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Signing Up" ]
+    p [] [
+        rawText "myPrayerJournal uses login services using Google or Microsoft accounts. The only information the "
+        rawText "application stores in its database is your user Id token it receives from these services, so there "
+        rawText "are no permissions you should have to accept from these provider other than establishing that you can "
+        rawText "log on with that account. Because of this, you’ll want to pick the same one each time; the "
+        rawText "tokens between the two accounts are different, even if you use the same e-mail address to log on to "
+        rawText "both."
+    ]
+]
+
+/// The "Your Prayer Journal" section
+let private yourJournal = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Your Prayer Journal" ]
+    p [] [
+        rawText "Your current requests will be presented in columns (usually three, but it could be more or less, "
+        rawText "depending on the size of your screen or device). Each request is in its own card, and the buttons at "
+        rawText "the top of each card apply to that request. The last line of each request also tells you how long it "
+        rawText "has been since anything has been done on that request. Any time you see something like “a few "
+        rawText "minutes ago,” you can hover over that to see the actual date/time the action was taken."
+    ]
+]
+
+/// The "Adding a Request" section
+let private addRequest = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Adding a Request" ]
+    p [] [
+        rawText "To add a request, click the “Add a New Request” button at the top of your journal. Then, "
+        rawText "enter the text of the request as you see fit; there is no right or wrong way, and you are the only "
+        rawText "person who will see the text you enter. When you save the request, it will go to the bottom of the "
+        rawText "list of requests."
+    ]
+]
+
+/// The "Setting Request Recurrence" section
+let private setRecurrence = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Setting Request Recurrence" ]
+    p [] [
+        rawText "When you add or update a request, you can choose whether requests go to the bottom of the journal "
+        rawText "once they have been marked “Prayed” or whether they will reappear after a delay. You can "
+        rawText "set recurrence in terms of hours, days, or weeks, but it cannot be longer than 365 days. If you "
+        rawText "decide you want a request to reappear sooner, you can skip the current delay; click the "
+        rawText "“Active” menu link, find the request in the list (likely near the bottom), and click the "
+        rawText "“Show Now” button."
+    ]
+]
+
+/// The "Praying for Requests" section
+let private praying = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Praying for Requests" ]
+    p [] [
+        rawText "The first button for each request has a checkmark icon; clicking this button will mark the request as "
+        rawText "“Prayed” and move it to the bottom of the list (or off, if you’ve set a recurrence "
+        rawText "period for the request). This allows you, if you’re praying through your requests, to start at "
+        rawText "the top left (with the request that it’s been the longest since you’ve prayed) and click "
+        rawText "the button as you pray; when the request move below or away, the next-least-recently-prayed request "
+        rawText "will take the top spot."
+    ]
+]
+
+/// The "Editing Requests" section
+let private editing = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Editing Requests" ]
+    p [] [
+        rawText "The second button for each request has a pencil icon. This allows you to edit the text of the "
+        rawText "request, pretty much the same way you entered it; it starts with the current text, and you can add to "
+        rawText "it, modify it, or completely replace it. By default, updates will go in with an “Updated” "
+        rawText "status; you have the option to also mark this update as “Prayed” or "
+        rawText "“Answered”. Answered requests will drop off the journal list."
+    ]
+]
+
+/// The "Adding Notes" section
+let private addNotes = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Adding Notes" ]
+    p [] [
+        rawText "The third button for each request has an icon that looks like a speech bubble with lines on it; this "
+        rawText "lets you record notes about the request. If there is something you want to record that doesn’t "
+        rawText "change the text of the request, this is the place to do it. For example, you may be praying for a "
+        rawText "long-term health issue, and that person tells you that their status is the same; or, you may want to "
+        rawText "record something God said to you while you were praying for that request."
+    ]
+]
+
+/// The "Snoozing Requests" section
+let private snoozing = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Snoozing Requests" ]
+    p [] [
+        rawText "There may be a time where a request does not need to appear. The fourth button, with the clock icon, "
+        rawText "allows you to snooze requests until the day you specify. Additionally, if you have any snoozed "
+        rawText "requests, a “Snoozed” menu item will appear next to the “Journal” one; this "
+        rawText "page allows you to see what requests are snoozed, and return them to your journal by canceling the "
+        rawText "snooze."
+    ]
+]
+
+/// The "Viewing a Request and Its History" section
+let private viewing = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Viewing a Request and Its History" ]
+    p [] [
+        rawText "myPrayerJournal tracks all of the actions related to a request; from the “Active” and "
+        rawText "“Answered” menu links (and “Snoozed”, if it’s showing), there is a "
+        rawText "“View Full Request” button. That page will show the current text of the request; how many "
+        rawText "times it has been marked as prayed; how long it has been an active request; and a log of all updates, "
+        rawText "prayers, and notes you have recorded. That log is listed from most recent to least recent; if you "
+        rawText "want to read it chronologically, press the “End” key on your keyboard and read it from "
+        rawText "the bottom up."
+    ]
+    p [] [
+        rawText "The “Active” link will show all requests that have not yet been marked answered, "
+        rawText "including snoozed and recurring requests. If requests are snoozed, or in a recurrence period off the "
+        rawText "journal, there will be a button where you can return the request to the list (either “Cancel "
+        rawText "Snooze” or “Show Now”). The “Answered” link shows all requests that "
+        rawText "have been marked answered. The “Snoozed” link only shows snoozed requests."
+    ]
+]
+
+/// The "Final Notes" section
+let private finalNotes = [
+    h3 [ _class "mb-3 mt-4" ] [ rawText "Final Notes" ]
+    ul [] [
+        li [] [
+            rawText "If you encounter errors, please "
+            a [ _href "https://git.bitbadger.solutions/bit-badger/myPrayerJournal/issues"; _target "_blank" ] [
+                rawText "file an issue"
+            ]; rawText " (or "
+            a [ _href "mailto:daniel@bitbadger.solutions?subject=myPrayerJournal+Issue" ] [ rawText "e-mail Daniel" ]
+            rawText " if you do not have an account on that server) with as much detail as possible. You can also "
+            rawText "provide suggestions, or browse the list of currently open issues."
+        ]
+        li [] [
+            rawText "Prayer requests and their history are securely backed up nightly along with other Bit Badger "
+            rawText "Solutions data."
+        ]
+        li [] [
+            rawText "Prayer changes things - most of all, the one doing the praying. I pray that this tool enables you "
+            rawText "to deepen and strengthen your prayer life."
+        ]
+    ]
+]
+
+/// The documentation page
+let index =
+    article [ _class "container mt-3" ] [
+        h2 [ _class "mb-3" ] [ rawText "Documentation" ]
+        yield! about
+        yield! signUp
+        yield! yourJournal
+        yield! addRequest
+        yield! setRecurrence
+        yield! praying
+        yield! editing
+        yield! addNotes
+        yield! snoozing
+        yield! viewing
+        yield! finalNotes
+    ]

src/MyPrayerJournal/Views/Helpers.fs

@@ -2,6 +2,7 @@
 [<AutoOpen>]
 module private MyPrayerJournal.Views.Helpers
   
+open Giraffe.Htmx
 open Giraffe.ViewEngine
 open Giraffe.ViewEngine.Htmx
 open MyPrayerJournal
@@ -10,7 +11,7 @@ open NodaTime
 /// Create a link that targets the `#top` element and pushes a URL to history
 let pageLink href attrs =
     attrs
-  |> List.append [ _href href; _hxBoost; _hxTarget "#top"; _hxSwap HxSwap.InnerHtml; _hxPushUrl ]
+    |> List.append [ _href href; _hxBoost; _hxTarget "#top"; _hxSwap HxSwap.InnerHtml; _hxPushUrl "true" ]
     |> a
 
 /// Create a Material icon
@@ -27,5 +28,15 @@ let noResults heading link buttonText text =
     ]
 
 /// Create a date with a span tag, displaying the relative date with the full date/time in the tooltip
-let relativeDate (date : Instant) now =
-  span [ _title (date.ToDateTimeOffset().ToString ("f", null)) ] [ Dates.formatDistance now date |> str ]
+let relativeDate (date : Instant) now (tz : DateTimeZone) =
+    span [ _title (date.InZone(tz).ToDateTimeOffset().ToString("f", null)) ] [ Dates.formatDistance now date |> str ]
+
+/// The version of myPrayerJournal
+let version =
+    let v = System.Reflection.Assembly.GetExecutingAssembly().GetName().Version
+    seq {
+        string v.Major
+        if v.Minor > 0 then
+            $".{v.Minor}"
+            if v.Revision > 0 then $".{v.Revision}"
+    } |> Seq.reduce (+)

src/MyPrayerJournal/Views/Journal.fs

@@ -1,60 +1,66 @@
 /// Views for journal pages and components
 module MyPrayerJournal.Views.Journal
 
+open Giraffe.Htmx
 open Giraffe.ViewEngine
 open Giraffe.ViewEngine.Accessibility
 open Giraffe.ViewEngine.Htmx
 open MyPrayerJournal
 
 /// Display a card for this prayer request
-let journalCard now req =
-  let reqId = RequestId.toString req.requestId
+let journalCard now tz req =
+    let reqId = RequestId.toString req.RequestId
     let spacer = span [] [ rawText " " ]
     div [ _class "col" ] [
         div [ _class "card h-100" ] [
             div [ _class "card-header p-0 d-flex"; _roleToolBar ] [
                 pageLink $"/request/{reqId}/edit" [ _class  "btn btn-secondary"; _title "Edit Request" ] [ icon "edit" ]
                 spacer
-        button [
-          _type     "button"
+                button [ _type     "button"
                          _class    "btn btn-secondary"
                          _title    "Add Notes"
                          _data     "bs-toggle" "modal"
                          _data     "bs-target" "#notesModal"
                          _hxGet    $"/components/request/{reqId}/add-notes"
                          _hxTarget "#notesBody"
-          _hxSwap   HxSwap.InnerHtml
-          ] [ icon "comment" ]
+                         _hxSwap   HxSwap.InnerHtml ] [
+                    icon "comment"
+                ]
                 spacer
-        button [
-          _type     "button"
+                button [ _type     "button"
                          _class    "btn btn-secondary"
                          _title    "Snooze Request"
                          _data     "bs-toggle" "modal"
                          _data     "bs-target" "#snoozeModal"
                          _hxGet    $"/components/request/{reqId}/snooze"
                          _hxTarget "#snoozeBody"
-          _hxSwap   HxSwap.InnerHtml
-          ] [ icon "schedule" ]
+                         _hxSwap   HxSwap.InnerHtml ] [
+                    icon "schedule"
+                ]
                 div [ _class "flex-grow-1" ] []
-        button [
-          _type    "button"
+                button [ _type    "button"
                          _class   "btn btn-success w-25"
                          _hxPatch $"/request/{reqId}/prayed"
-          _title   "Mark as Prayed"
-          ] [ icon "done" ]
+                         _title   "Mark as Prayed" ] [
+                    icon "done"
+                ]
             ]
             div [ _class "card-body" ] [
-        p [ _class "request-text" ] [ str req.text ]
+                p [ _class "request-text" ] [ str req.Text ]
             ]
             div [ _class "card-footer text-end text-muted px-1 py-0" ] [
-        em [] [ str "last activity "; relativeDate req.asOf now ]
+                em [] [
+                    match req.LastPrayed with
+                    | Some dt -> str "last prayed ";   relativeDate dt       now tz
+                    | None    -> str "last activity "; relativeDate req.AsOf now tz
+                ]
             ]
         ]
     ]
 
 /// The journal loading page
-let journal user = article [ _class "container-fluid mt-3" ] [
+let journal user =
+    article [ _class "container-fluid mt-3" ] [
         h2 [ _class "pb-3" ] [
             str user
             match user with "Your" -> () | _ -> rawText "’s"
@@ -66,13 +72,11 @@ let journal user = article [ _class "container-fluid mt-3" ] [
         p [ _hxGet "/components/journal-items"; _hxSwap HxSwap.OuterHtml; _hxTrigger HxTrigger.Load ] [
             rawText "Loading your prayer journal…"
         ]
-  div [
-    _id             "notesModal"
+        div [ _id             "notesModal"
               _class          "modal fade"
               _tabindex       "-1"
               _ariaLabelledBy "nodesModalLabel"
-    _ariaHidden     "true"
-    ] [
+              _ariaHidden     "true" ] [
             div [ _class "modal-dialog modal-dialog-scrollable" ] [
                 div [ _class "modal-content" ] [
                     div [ _class "modal-header" ] [
@@ -81,20 +85,21 @@ let journal user = article [ _class "container-fluid mt-3" ] [
                     ]
                     div [ _class "modal-body"; _id "notesBody" ] [ ]
                     div [ _class "modal-footer" ] [
-          button [ _type "button"; _id "notesDismiss"; _class "btn btn-secondary"; _data "bs-dismiss" "modal" ] [
+                        button [ _type  "button"
+                                 _id    "notesDismiss"
+                                 _class "btn btn-secondary"
+                                 _data  "bs-dismiss" "modal" ] [
                             str "Close"
                         ]
                     ]
                 ]
             ]
         ]
-  div [
-    _id             "snoozeModal"
+        div [ _id             "snoozeModal"
               _class          "modal fade"
               _tabindex       "-1"
               _ariaLabelledBy "snoozeModalLabel"
-    _ariaHidden     "true"
-    ] [
+              _ariaHidden     "true" ] [
             div [ _class "modal-dialog modal-sm" ] [
                 div [ _class "modal-content" ] [
                     div [ _class "modal-header" ] [
@@ -103,7 +108,10 @@ let journal user = article [ _class "container-fluid mt-3" ] [
                     ]
                     div [ _class "modal-body"; _id "snoozeBody" ] [ ]
                     div [ _class "modal-footer" ] [
-          button [ _type "button"; _id "snoozeDismiss"; _class "btn btn-secondary"; _data "bs-dismiss" "modal" ] [
+                        button [ _type "button"
+                                 _id "snoozeDismiss"
+                                 _class "btn btn-secondary"
+                                 _data "bs-dismiss" "modal" ] [
                             str "Close"
                         ]
                       ]
@@ -113,7 +121,7 @@ let journal user = article [ _class "container-fluid mt-3" ] [
     ]
 
 /// The journal items
-let journalItems now items =
+let journalItems now tz items =
     match items |> List.isEmpty with
     | true ->
         noResults "No Active Requests" "/request/new/edit" "Add a Request" [
@@ -122,27 +130,24 @@ let journalItems now items =
         ]
     | false ->
         items
-      |> List.map (journalCard now)
-      |> section [
-          _id       "journalItems"
+        |> List.map (journalCard now tz)
+        |> section [ _id        "journalItems"
                      _class     "row row-cols-1 row-cols-md-2 row-cols-lg-3 row-cols-xl-4 g-3"
                      _hxTarget  "this"
                      _hxSwap    HxSwap.OuterHtml
-          ]
+                     _ariaLabel "Prayer Requests" ]
 
 /// The notes edit modal body
 let notesEdit requestId =
     let reqId = RequestId.toString requestId
     [   form [ _hxPost $"/request/{reqId}/note" ] [
             div [ _class "form-floating pb-3" ] [
-        textarea [
-          _id          "notes"
+                textarea [ _id          "notes"
                            _name        "notes"
                            _class       "form-control"
                            _style       "min-height: 8rem;"
                            _placeholder "Notes"
-          _autofocus;  _required
-          ] [ ]
+                           _autofocus;  _required ] [ ]
                 label [ _for "notes" ] [ str "Notes" ]
             ]
             p [ _class "text-end" ] [ button [ _type "submit"; _class "btn btn-primary" ] [ str "Add Notes" ] ]
@@ -150,13 +155,13 @@ let notesEdit requestId =
         hr [ _style "margin: .5rem -1rem" ]
         div [ _id "priorNotes" ] [
             p [ _class "text-center pt-3" ] [
-        button [
-          _type     "button"
+                button [ _type     "button"
                          _class    "btn btn-secondary"
                          _hxGet    $"/components/request/{reqId}/notes"
                          _hxSwap   HxSwap.OuterHtml
-          _hxTarget "#priorNotes"
-          ] [str "Load Prior Notes" ]
+                         _hxTarget "#priorNotes" ] [
+                    str "Load Prior Notes"
+                ]
             ]
         ]
     ]
@@ -164,11 +169,9 @@ let notesEdit requestId =
 /// The snooze edit form
 let snooze requestId =
     let today = System.DateTime.Today.ToString "yyyy-MM-dd"
-  form [
-    _hxPatch  $"/request/{RequestId.toString requestId}/snooze"
+    form [ _hxPatch  $"/request/{RequestId.toString requestId}/snooze"
            _hxTarget "#journalItems"
-    _hxSwap   HxSwap.OuterHtml
-    ] [
+           _hxSwap   HxSwap.OuterHtml ] [
         div [ _class "form-floating pb-3" ] [
             input [ _type "date"; _id "until"; _name "until"; _class "form-control"; _min today; _required ]
             label [ _for "until" ] [ str "Until" ]

src/MyPrayerJournal/Views/Layout.fs

@@ -1,151 +1,125 @@
 /// Layout / home views
 module MyPrayerJournal.Views.Layout
 
-// fsharplint:disable RecordFieldNames
-
 open Giraffe.ViewEngine
 open Giraffe.ViewEngine.Accessibility
 
 /// The data needed to render a page-level view
-type PageRenderContext = {
-  /// Whether the user is authenticated
-  isAuthenticated : bool
+type PageRenderContext =
+    { /// Whether the user is authenticated
+      IsAuthenticated: bool
+      
       /// Whether the user has snoozed requests
-  hasSnoozed      : bool
+      HasSnoozed: bool
+      
       /// The current URL
-  currentUrl      : string
+      CurrentUrl: string
+      
       /// The title for the page to be rendered
-  pageTitle       : string
+      PageTitle: string
+      
       /// The content of the page
-  content         : XmlNode
-  }
+      Content: XmlNode }
 
 /// The home page
-let home = article [ _class "container mt-3" ] [
-  p [] [ rawText " " ]
-  p [] [
-    str "myPrayerJournal is a place where individuals can record their prayer requests, record that they prayed for "
-    str "them, update them as God moves in the situation, and record a final answer received on that request. It also "
-    str "allows individuals to review their answered prayers."
-    ]
-  p [] [
-    str "This site is open and available to the general public. To get started, simply click the "
-    rawText "“Log On” link above, and log on with either a Microsoft or Google account. You can also "
-    rawText "learn more about the site at the “Docs” link, also above."
-    ]
-  ]
+let home =
+    article [ _class "container mt-3" ]
+        [ p [] [ rawText " " ]
+          p []
+            [ str "myPrayerJournal is a place where individuals can record their prayer requests, record that they "
+              str "prayed for them, update them as God moves in the situation, and record a final answer received on "
+              str "that request. It also allows individuals to review their answered prayers." ]
+          p []
+            [ str "This site is open and available to the general public. To get started, simply click the "
+              rawText "“Log On” link above, and log on with either a Microsoft or Google account. You can "
+              rawText "also learn more about the site at the “Docs” link, also above." ] ]
 
 /// The default navigation bar, which will load the items on page load, and whenever a refresh event occurs
 let private navBar ctx =
-  nav [ _class "navbar navbar-dark"; _roleNavigation ] [
-    div [ _class "container-fluid" ] [
-      pageLink "/" [ _class "navbar-brand" ] [
-        span [ _class "m" ] [ str "my" ]
+    nav [ _class "navbar navbar-dark"; _roleNavigation ]
+        [ div [ _class "container-fluid" ]
+            [ pageLink
+                "/" [ _class "navbar-brand" ]
+                [ span [ _class "m" ] [ str "my" ]
                   span [ _class "p" ] [ str "Prayer" ]
-        span [ _class "j" ] [ str "Journal" ]
-      ]
+                  span [ _class "j" ] [ str "Journal" ] ]
               seq {
                 let navLink (matchUrl : string) =
-          match ctx.currentUrl.StartsWith matchUrl with true -> [ _class "is-active-route" ] | false -> []
+                    match ctx.CurrentUrl.StartsWith matchUrl with true -> [ _class "is-active-route" ] | false -> []
                     |> pageLink matchUrl
-        match ctx.isAuthenticated with
-        | true ->
+                if ctx.IsAuthenticated then
                     li [ _class "nav-item" ] [ navLink "/journal" [ str "Journal" ] ]
                     li [ _class "nav-item" ] [ navLink "/requests/active" [ str "Active" ] ]
-            if ctx.hasSnoozed then li [ _class "nav-item" ] [ navLink "/requests/snoozed" [ str "Snoozed" ] ]
+                    if ctx.HasSnoozed then li [ _class "nav-item" ] [ navLink "/requests/snoozed" [ str "Snoozed" ] ]
                     li [ _class "nav-item" ] [ navLink "/requests/answered" [ str "Answered" ] ]
                     li [ _class "nav-item" ] [ a [ _href "/user/log-off" ] [ str "Log Off" ] ]
-        | false -> li [ _class "nav-item"] [ a [ _href "/user/log-on" ] [ str "Log On" ] ]
-        li [ _class "nav-item" ] [
-          a [ _href "https://docs.prayerjournal.me"; _target "_blank"; _rel "noopener" ] [ str "Docs" ]
-          ]
+                else li [ _class "nav-item"] [ a [ _href "/user/log-on" ] [ str "Log On" ] ]
+                li [ _class "nav-item" ] [ navLink "/docs" [ str "Docs" ] ]
               }
               |> List.ofSeq
-      |> ul [ _class "navbar-nav me-auto d-flex flex-row" ]
-      ]
-    ]
+              |> ul [ _class "navbar-nav me-auto d-flex flex-row" ] ] ]
 
 /// The title tag with the application name appended
-let titleTag ctx = title [] [ str ctx.pageTitle; rawText " « myPrayerJournal" ]
+let titleTag ctx =
+    title [] [ rawText ctx.PageTitle; rawText " « myPrayerJournal" ]
 
 /// The HTML `head` element
 let htmlHead ctx =
-  head [ _lang "en" ] [
-    meta [ _name "viewport"; _content "width=device-width, initial-scale=1" ]
+    head [ _lang "en" ]
+        [ meta [ _name "viewport"; _content "width=device-width, initial-scale=1" ]
           meta [ _name "description"; _content "Online prayer journal - free w/Google or Microsoft account" ]
           titleTag ctx
-    link [
-      _href        "https://cdn.jsdelivr.net/npm/bootstrap@5.0.2/dist/css/bootstrap.min.css"
+          link [ _href        "https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/css/bootstrap.min.css"
                  _rel         "stylesheet"
-      _integrity   "sha384-EVSTQN3/azprG1Anm3QDgpJLIm9Nao0Yz1ztcQTwFspd3yD65VohhpuuCOmLASjC"
-      _crossorigin "anonymous"
-      ]
+                 _integrity   "sha384-T3c6CoIi6uLrA9TneNEoa7RxnatzjcDSCmG1MXxSR1GAsXEV/Dwwykc2MPK8M2HN"
+                 _crossorigin "anonymous" ]
           link [ _href "https://fonts.googleapis.com/icon?family=Material+Icons"; _rel "stylesheet" ]
-    link [ _href "/style/style.css"; _rel "stylesheet" ]
-    ]
+          link [ _href "/style/style.css"; _rel "stylesheet" ] ]
 
 /// Element used to display toasts
 let toaster =
-  div [ _ariaLive "polite"; _ariaAtomic "true"; _id "toastHost" ] [
-    div [ _class "toast-container position-absolute p-3 bottom-0 end-0"; _id "toasts" ] []
-  ]
+    div [ _ariaLive "polite"; _ariaAtomic "true"; _id "toastHost" ]
+        [ div [ _class "toast-container position-absolute p-3 bottom-0 end-0"; _id "toasts" ] [] ]
 
 /// The page's `footer` element
 let htmlFoot =
-  footer [ _class "container-fluid" ] [
-    p [ _class "text-muted text-end" ] [
-      str "myPrayerJournal v3"
+    footer [ _class "container-fluid" ]
+        [ p [ _class "text-muted text-end" ]
+            [ str $"myPrayerJournal {version}"
               br []
-      em [] [
-        small [] [
-          pageLink "/legal/privacy-policy" [] [ str "Privacy Policy" ]
+              em []
+                  [ small []
+                        [ pageLink "/legal/privacy-policy" [] [ str "Privacy Policy" ]
                           rawText " • "
                           pageLink "/legal/terms-of-service" [] [ str "Terms of Service" ]
                           rawText " • "
-          a [ _href "https://github.com/bit-badger/myprayerjournal"; _target "_blank"; _rel "noopener" ] [
-            str "Developed"
-            ]
+                          a [ _href "https://git.bitbadger.solutions/bit-badger/myPrayerJournal"
+                              _target "_blank"
+                              _rel "noopener" ] [ str "Developed" ]
                           str " and hosted by "
-          a [ _href "https://bitbadger.solutions"; _target "_blank"; _rel "noopener" ] [ str "Bit Badger Solutions" ]
-          ]
-        ]
-      ]
-    script [
-      _src         "https://unpkg.com/htmx.org@1.5.0"
-      _integrity   "sha384-oGA+prIp5Vchu6we2YkI51UtVzN9Jpx2Z7PnR1I78PnZlN8LkrCT4lqqqmDkyrvI"
-      _crossorigin "anonymous"
-      ] []
-    script [] [
-      rawText "if (!htmx) document.write('<script src=\"/script/htmx-1.5.0.min.js\"><\/script>')"
-      ]
-    script [
-      _async
-      _src         "https://cdn.jsdelivr.net/npm/bootstrap@5.0.2/dist/js/bootstrap.bundle.min.js"
-      _integrity   "sha384-MrcW6ZMFYlzcLA8Nl+NtUVF0sA7MsXsP1UyJoMp4YLEuNSfAP+JcXn/tWtIaxVXM"
-      _crossorigin "anonymous"
-      ] []
-    script [] [
-      rawText "setTimeout(function () { "
+                          a   [ _href "https://bitbadger.solutions"; _target "_blank"; _rel "noopener" ]
+                              [ str "Bit Badger Solutions" ] ] ] ]
+          Htmx.Script.minified
+          script [] [ rawText "if (!htmx) document.write('<script src=\"/script/htmx.min.js\"><\/script>')" ]
+          script [ _async
+                   _src         "https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js"
+                   _integrity   "sha384-C6RzsynM9kWDrMNeT87bh95OGNyZPhcTNXj1NW7RuBCsyN/o0jlpcV8Qyq46cDfL"
+                   _crossorigin "anonymous" ] []
+          script []
+              [ rawText "setTimeout(function () { "
                 rawText "if (!bootstrap) document.write('<script src=\"/script/bootstrap.bundle.min.js\"><\/script>') "
-      rawText "}, 2000)"
-      ]
-    script [ _src "/script/mpj.js" ] []
-    ]
+                rawText "}, 2000)" ]
+          script [ _src "/script/mpj.js" ] [] ]
 
 /// Create the full view of the page
 let view ctx =
-  html [ _lang "en" ] [
-    htmlHead ctx
-    body [] [
-      section [ _id "top" ] [ navBar ctx; main [ _roleMain ] [ ctx.content ] ]
+    html [ _lang "en" ]
+        [ htmlHead ctx
+          body []
+            [ section [ _id "top"; _ariaLabel "Top navigation" ] [ navBar ctx; main [ _roleMain ] [ ctx.Content ] ]
               toaster
-      htmlFoot
-      ]
-    ]
+              htmlFoot ] ]
 
 /// Create a partial view
 let partial ctx =
-  html [ _lang "en" ] [
-    head [] [ titleTag ctx ]
-    body [] [ navBar ctx; main [ _roleMain ] [ ctx.content ] ]
-    ]
+    html [ _lang "en" ] [ head [] [ titleTag ctx ]; body [] [ navBar ctx; main [ _roleMain ] [ ctx.Content ] ] ]

src/MyPrayerJournal/Views/Legal.fs

@@ -4,23 +4,26 @@ module MyPrayerJournal.Views.Legal
 open Giraffe.ViewEngine
 
 /// View for the "Privacy Policy" page
-let privacyPolicy = article [ _class "container mt-3" ] [
+let privacyPolicy =
+    article [ _class "container mt-3" ] [
         h2 [ _class "mb-2" ] [ str "Privacy Policy" ]
         h6 [ _class "text-muted pb-3" ] [ str "as of May 21"; sup [] [ str "st"]; str ", 2018" ]
         p [] [
-    str "The nature of the service is one where privacy is a must. The items below will help you understand the data "
-    str "we collect, access, and store on your behalf as you use this service."
+            str "The nature of the service is one where privacy is a must. The items below will help you understand "
+            str "the data we collect, access, and store on your behalf as you use this service."
         ]
         div [ _class "card" ] [
             div [ _class "list-group list-group-flush" ] [
                 div [ _class "list-group-item"] [
                     h3 [] [ str "Third Party Services" ]
                     p [ _class "card-text" ] [
-          str "myPrayerJournal utilizes a third-party authentication and identity provider. You should familiarize "
-          str "yourself with the privacy policy for "
+                        str "myPrayerJournal utilizes a third-party authentication and identity provider. You should "
+                        str "familiarize yourself with the privacy policy for "
                         a [ _href "https://auth0.com/privacy"; _target "_blank" ] [ str "Auth0" ]
                         str ", as well as your chosen provider ("
-          a [ _href "https://privacy.microsoft.com/en-us/privacystatement"; _target "_blank" ] [ str "Microsoft"]
+                        a [ _href "https://privacy.microsoft.com/en-us/privacystatement"; _target "_blank" ] [
+                            str "Microsoft"
+                        ]
                         str " or "
                         a [ _href "https://policies.google.com/privacy"; _target "_blank" ] [ str "Google" ]
                         str ")."
@@ -31,22 +34,23 @@ let privacyPolicy = article [ _class "container mt-3" ] [
                     h4 [] [ str "Identifying Data" ]
                     ul [] [
                         li [] [
-            rawText "The only identifying data myPrayerJournal stores is the subscriber (“sub”) field from "
-            str "the token we receive from Auth0, once you have signed in through their hosted service. All "
-            str "information is associated with you via this field."
+                            str "The only identifying data myPrayerJournal stores is the subscriber "
+                            rawText "(“sub”) field from the token we receive from Auth0, once you have "
+                            str "signed in through their hosted service. All information is associated with you via "
+                            str "this field."
                         ]
                         li [] [
-            str "While you are signed in, within your browser, the service has access to your first and last names, "
-            str "along with a URL to the profile picture (provided by your selected identity provider). This "
-            rawText "information is not transmitted to the server, and is removed when “Log Off” is "
-            str "clicked."
+                            str "While you are signed in, within your browser, the service has access to your first "
+                            str "and last names, along with a URL to the profile picture (provided by your selected "
+                            str "identity provider). This information is not transmitted to the server, and is removed "
+                            rawText "when “Log Off” is clicked."
                         ]
                     ]
                     h4 [] [ str "User Provided Data" ]
                     ul [ _class "mb-0" ] [
                         li [] [
-            str "myPrayerJournal stores the information you provide, including the text of prayer requests, updates, "
-            str "and notes; and the date/time when certain actions are taken."
+                            str "myPrayerJournal stores the information you provide, including the text of prayer "
+                            str "requests, updates, and notes; and the date/time when certain actions are taken."
                         ]
                     ]
                 ]
@@ -54,35 +58,38 @@ let privacyPolicy = article [ _class "container mt-3" ] [
                     h3 [] [ str "How Your Data Is Accessed / Secured" ]
                     ul [ _class "mb-0" ] [
                         li [] [
-            str "Your provided data is returned to you, as required, to display your journal or your answered "
-            str "requests. On the server, it is stored in a controlled-access database."
+                            str "Your provided data is returned to you, as required, to display your journal or your "
+                            str "answered requests. On the server, it is stored in a controlled-access database."
                         ]
                         li [] [
-            str "Your data is backed up, along with other Bit Badger Solutions hosted systems, in a rolling manner; "
-            str "backups are preserved for the prior 7 days, and backups from the 1"
+                            str "Your data is backed up, along with other Bit Badger Solutions hosted systems, in a "
+                            str "rolling manner; backups are preserved for the prior 7 days, and backups from the 1"
                             sup [] [ str "st" ]
                             str " and 15"
                             sup [] [ str "th" ]
-            str " are preserved for 3 months. These backups are stored in a private cloud data repository."
+                            str " are preserved for 3 months. These backups are stored in a private cloud data "
+                            str "repository."
                         ]
                         li [] [
-            str "The data collected and stored is the absolute minimum necessary for the functionality of the service. "
-            rawText "There are no plans to “monetize” this service, and storing the minimum amount of "
-            str "information means that the data we have is not interesting to purchasers (or those who may have more "
-            str "nefarious purposes)."
+                            str "The data collected and stored is the absolute minimum necessary for the functionality "
+                            rawText "of the service. There are no plans to “monetize” this service, and "
+                            str "storing the minimum amount of information means that the data we have is not "
+                            str "interesting to purchasers (or those who may have more nefarious purposes)."
                         ]
                         li [] [
-            str "Access to servers and backups is strictly controlled and monitored for unauthorized access attempts."
+                            str "Access to servers and backups is strictly controlled and monitored for unauthorized "
+                            str "access attempts."
                         ]
                     ]
                 ]
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "Removing Your Data" ]
                     p [ _class "card-text" ] [
-          str "At any time, you may choose to discontinue using this service. Both Microsoft and Google provide ways "
-          str "to revoke access from this application. However, if you want your data removed from the database, "
-          str "please contact daniel at bitbadger.solutions (via e-mail, replacing at with @) prior to doing so, to "
-          str "ensure we can determine which subscriber ID belongs to you."
+                        str "At any time, you may choose to discontinue using this service. Both Microsoft and Google "
+                        str "provide ways to revoke access from this application. However, if you want your data "
+                        str "removed from the database, please contact daniel at bitbadger.solutions (via e-mail, "
+                        str "replacing at with @) prior to doing so, to ensure we can determine which subscriber ID "
+                        str "belongs to you."
                     ]
                 ]
             ]
@@ -90,7 +97,8 @@ let privacyPolicy = article [ _class "container mt-3" ] [
     ]
 
 /// View for the "Terms of Service" page
-let termsOfService = article [ _class "container mt-3" ] [
+let termsOfService =
+    article [ _class "container mt-3" ] [
         h2 [ _class "mb-2" ] [ str "Terms of Service" ]
         h6 [ _class "text-muted pb-3"] [ str "as of May 21"; sup [] [ str "st" ]; str ", 2018" ]
         div [ _class "card" ] [
@@ -98,17 +106,17 @@ let termsOfService = article [ _class "container mt-3" ] [
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "1. Acceptance of Terms" ]
                     p [ _class "card-text" ] [
-          str "By accessing this web site, you are agreeing to be bound by these Terms and Conditions, and that you "
-          str "are responsible to ensure that your use of this site complies with all applicable laws. Your continued "
-          str "use of this site implies your acceptance of these terms."
+                        str "By accessing this web site, you are agreeing to be bound by these Terms and Conditions, "
+                        str "and that you are responsible to ensure that your use of this site complies with all "
+                        str "applicable laws. Your continued use of this site implies your acceptance of these terms."
                     ]
                 ]
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "2. Description of Service and Registration" ]
                     p [ _class "card-text" ] [
-          str "myPrayerJournal is a service that allows individuals to enter and amend their prayer requests. It "
-          str "requires no registration by itself, but access is granted based on a successful login with an external "
-          str "identity provider. See "
+                        str "myPrayerJournal is a service that allows individuals to enter and amend their prayer "
+                        str "requests. It requires no registration by itself, but access is granted based on a "
+                        str "successful login with an external identity provider. See "
                         pageLink "/legal/privacy-policy" [] [ str "our privacy policy" ]
                         str " for details on how that information is accessed and stored."
                     ]
@@ -116,11 +124,13 @@ let termsOfService = article [ _class "container mt-3" ] [
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "3. Third Party Services" ]
                     p [ _class "card-text" ] [
-          str "This service utilizes a third-party service provider for identity management. Review the terms of "
-          str "service for "
+                        str "This service utilizes a third-party service provider for identity management. Review the "
+                        str "terms of service for "
                         a [ _href "https://auth0.com/terms"; _target "_blank" ] [ str "Auth0"]
                         str ", as well as those for the selected authorization provider ("
-          a [ _href "https://www.microsoft.com/en-us/servicesagreement"; _target "_blank" ] [ str "Microsoft"]
+                        a [ _href "https://www.microsoft.com/en-us/servicesagreement"; _target "_blank" ] [
+                            str "Microsoft"
+                        ]
                         str " or "
                         a [ _href "https://policies.google.com/terms"; _target "_blank" ] [ str "Google" ]
                         str ")."
@@ -129,17 +139,17 @@ let termsOfService = article [ _class "container mt-3" ] [
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "4. Liability" ]
                     p [ _class "card-text" ] [
-          rawText "This service is provided “as is”, and no warranty (express or implied) exists. The "
-          str "service and its developers may not be held liable for any damages that may arise through the use of "
-          str "this service."
+                        rawText "This service is provided “as is”, and no warranty (express or implied) "
+                        str "exists. The service and its developers may not be held liable for any damages that may "
+                        str "arise through the use of this service."
                     ]
                 ]
                 div [ _class "list-group-item" ] [
                     h3 [] [ str "5. Updates to Terms" ]
                     p [ _class "card-text" ] [
-          str "These terms and conditions may be updated at any time, and this service does not have the capability to "
-          str "notify users when these change. The date at the top of the page will be updated when any of the text of "
-          str "these terms is updated."
+                        str "These terms and conditions may be updated at any time, and this service does not have the "
+                        str "capability to notify users when these change. The date at the top of the page will be "
+                        str "updated when any of the text of these terms is updated."
                     ]
                 ]
             ]
@@ -150,4 +160,3 @@ let termsOfService = article [ _class "container mt-3" ] [
             str " to learn how we handle your data."
         ]
     ]
-

src/MyPrayerJournal/Views/Request.fs

@@ -1,108 +1,109 @@
 /// Views for request pages and components
 module MyPrayerJournal.Views.Request
 
+open Giraffe.Htmx
 open Giraffe.ViewEngine
 open Giraffe.ViewEngine.Htmx
 open MyPrayerJournal
 open NodaTime
-open System
 
 /// Create a request within the list
-let reqListItem now req =
-  let reqId      = RequestId.toString req.requestId
-  let isAnswered = req.lastStatus = Answered
-  let isSnoozed  = req.snoozedUntil > now
-  let isPending  = (not isSnoozed) && req.showAfter > now
+let reqListItem now tz req =
+    let isFuture instant = defaultArg (instant |> Option.map (fun it -> it > now)) false
+    let reqId      = RequestId.toString req.RequestId
+    let isAnswered = req.LastStatus = Answered
+    let isSnoozed  = isFuture req.SnoozedUntil
+    let isPending  = (not isSnoozed) && isFuture req.ShowAfter
     let btnClass   = _class "btn btn-light mx-2"
     let restoreBtn (link : string) title =
         button [ btnClass; _hxPatch $"/request/{reqId}/{link}"; _title title ] [ icon "restore" ]
-  div [ _class "list-group-item px-0 d-flex flex-row align-items-start"; _hxTarget "this"; _hxSwap HxSwap.OuterHtml ] [
+    div [ _class "list-group-item px-0 d-flex flex-row align-items-start"
+          _hxTarget "this"
+          _hxSwap HxSwap.OuterHtml ] [
         pageLink $"/request/{reqId}/full" [ btnClass; _title "View Full Request" ] [ icon "description" ]
-    match isAnswered with
-    | true  -> ()
-    | false -> pageLink $"/request/{reqId}/edit" [ btnClass; _title "Edit Request" ] [ icon "edit" ]
-    match true with
-    | _ when isSnoozed -> restoreBtn "cancel-snooze" "Cancel Snooze"
-    | _ when isPending -> restoreBtn "show"          "Show Now"
-    | _ -> ()
+        if not isAnswered then pageLink $"/request/{reqId}/edit" [ btnClass; _title "Edit Request" ] [ icon "edit" ]
+        if   isSnoozed then restoreBtn "cancel-snooze" "Cancel Snooze"
+        elif isPending then restoreBtn "show"          "Show Now"
         p [ _class "request-text mb-0" ] [
-      str req.text
-      match isSnoozed || isPending || isAnswered with
-      | true ->
+            str req.Text
+            if isSnoozed || isPending || isAnswered then
                 br []
                 small [ _class "text-muted" ] [
-            match () with
-            | _ when isSnoozed   -> [ str "Snooze expires ";       relativeDate req.snoozedUntil now ]
-            | _ when isPending   -> [ str "Request appears next "; relativeDate req.showAfter    now ]
-            | _ (* isAnswered *) -> [ str "Answered ";             relativeDate req.asOf         now ]
+                    if   isSnoozed then   [ str "Snooze expires ";       relativeDate req.SnoozedUntil.Value now tz ]
+                    elif isPending then   [ str "Request appears next "; relativeDate req.ShowAfter.Value    now tz ]
+                    else (* isAnswered *) [ str "Answered ";             relativeDate req.AsOf               now tz ]
                     |> em []
                 ]
-      | false -> ()
           ]
     ]
 
 /// Create a list of requests
-let reqList now reqs =
+let reqList now tz reqs =
     reqs
-  |> List.map (reqListItem now)
+    |> List.map (reqListItem now tz)
     |> div [ _class "list-group" ]
 
 /// View for Active Requests page
-let active now reqs = article [ _class "container mt-3" ] [
+let active now tz reqs =
+    article [ _class "container mt-3" ] [
         h2 [ _class "pb-3" ] [ str "Active Requests" ]
-  match reqs |> List.isEmpty with
-  | true ->
+        if List.isEmpty reqs then
             noResults "No Active Requests" "/journal" "Return to your journal"
                 [ str "Your prayer journal has no active requests" ]
-  | false -> reqList now reqs
+        else reqList now tz reqs
     ]
 
 /// View for Answered Requests page
-let answered now reqs = article [ _class "container mt-3" ] [
+let answered now tz reqs =
+    article [ _class "container mt-3" ] [
         h2 [ _class "pb-3" ] [ str "Answered Requests" ]
-  match reqs |> List.isEmpty with
-  | true ->
-      noResults "No Active Requests" "/journal" "Return to your journal" [
-        rawText "Your prayer journal has no answered requests; once you have marked one as “Answered”, "
-        str "it will appear here"
+        if List.isEmpty reqs then
+            noResults "No Answered Requests" "/journal" "Return to your journal" [
+                str "Your prayer journal has no answered requests; once you have marked one as "
+                rawText "“Answered”, it will appear here"
             ]
-  | false -> reqList now reqs
+        else reqList now tz reqs
     ]
 
 /// View for Snoozed Requests page
-let snoozed now reqs = article [ _class "container mt-3" ] [
+let snoozed now tz reqs =
+    article [ _class "container mt-3" ] [
         h2 [ _class "pb-3" ] [ str "Snoozed Requests" ]
-  reqList now reqs
+        reqList now tz reqs
     ]
 
 /// View for Full Request page
-let full (clock : IClock) (req : Request) =
+let full (clock : IClock) tz (req : Request) =
     let now = clock.GetCurrentInstant()
     let answered =
-    req.history
-    |> List.filter RequestAction.isAnswered
-    |> List.tryHead
-    |> Option.map (fun x -> x.asOf)
-  let prayed = (req.history |> List.filter RequestAction.isPrayed |> List.length).ToString "N0"
+        req.History
+        |> Seq.ofList
+        |> Seq.filter History.isAnswered
+        |> Seq.tryHead
+        |> Option.map (_.AsOf)
+    let prayed = (req.History |> List.filter History.isPrayed |> List.length).ToString "N0"
     let daysOpen =
         let asOf = defaultArg answered now
-    ((asOf - (req.history |> List.filter RequestAction.isCreated |> List.head).asOf).TotalDays |> int).ToString "N0"
+        ((asOf - (req.History |> List.filter History.isCreated |> List.head).AsOf).TotalDays |> int).ToString "N0"
     let lastText =
-    req.history
-    |> List.filter (fun h -> Option.isSome h.text)
-    |> List.sortByDescending (fun h -> h.asOf)
-    |> List.map (fun h -> Option.get h.text)
-    |> List.head
+        req.History
+        |> Seq.ofList
+        |> Seq.filter (fun h -> Option.isSome h.Text)
+        |> Seq.sortByDescending (_.AsOf)
+        |> Seq.map (fun h -> Option.get h.Text)
+        |> Seq.head
     // The history log including notes (and excluding the final entry for answered requests)
     let log =
-    let toDisp (h : History) = {| asOf = h.asOf; text = h.text; status = RequestAction.toString h.status |}
+        let toDisp (h : History) = {| asOf = h.AsOf; text = h.Text; status = RequestAction.toString h.Status |}
         let all =
-      req.notes
-      |> List.map (fun n -> {| asOf = n.asOf; text = Some n.notes; status = "Notes" |})
-      |> List.append (req.history |> List.map toDisp)
-      |> List.sortByDescending (fun it -> it.asOf)
+            req.Notes
+            |> Seq.ofList
+            |> Seq.map (fun n -> {| asOf = n.AsOf; text = Some n.Notes; status = "Notes" |})
+            |> Seq.append (req.History |> List.map toDisp)
+            |> Seq.sortByDescending (_.asOf)
+            |> List.ofSeq
         // Skip the first entry for answered requests; that info is already displayed
-    match answered with Some _ -> all |> List.skip 1 | None -> all
+        match answered with Some _ -> all.Tail | None -> all
     article [ _class "container mt-3" ] [
         div [_class "card" ] [
             h5 [ _class "card-header" ] [ str "Full Prayer Request" ]
@@ -113,15 +114,16 @@ let full (clock : IClock) (req : Request) =
                         str "Answered "
                         date.ToDateTimeOffset().ToString("D", null) |> str
                         str " ("
-              relativeDate date now
+                        relativeDate date now tz
                         rawText ") • "
                     | None -> ()
-          sprintf "Prayed %s times • Open %s days" prayed daysOpen |> rawText
+                    rawText $"Prayed %s{prayed} times • Open %s{daysOpen} days"
                 ]
                 p [ _class "card-text" ] [ str lastText ]
             ]
             log
-      |> List.map (fun it -> li [ _class "list-group-item" ] [
+            |> List.map (fun it ->
+                li [ _class "list-group-item" ] [
                     p [ _class "m-0" ] [
                         str it.status
                         rawText "  "
@@ -142,40 +144,45 @@ let edit (req : JournalRequest) returnTo isNew =
         | "active"          -> "/requests/active"
         | "snoozed"         -> "/requests/snoozed"
         | _ (* "journal" *) -> "/journal"
+    let recurCount =
+        match req.Recurrence with
+        | Immediate -> None
+        | Hours   h -> Some h
+        | Days    d -> Some d
+        | Weeks   w -> Some w
+        |> Option.map string
+        |> Option.defaultValue ""
     article [ _class "container" ] [
         h2 [ _class "pb-3" ] [ (match isNew with true -> "Add" | false -> "Edit") |> strf "%s Prayer Request" ]
-    form [
-      _hxBoost
+        form [ _hxBoost
                _hxTarget  "#top"
-      _hxPushUrl
-      "/request" |> match isNew with true -> _hxPost | false -> _hxPatch
-      ] [
-      input [
-        _type  "hidden"
+               _hxPushUrl "true"
+               "/request" |> match isNew with true -> _hxPost | false -> _hxPatch ] [
+            input [ _type  "hidden"
                     _name  "requestId"
-        _value (match isNew with true -> "new" | false -> RequestId.toString req.requestId)
-        ]
+                    _value (match isNew with true -> "new" | false -> RequestId.toString req.RequestId) ]
             input [ _type "hidden"; _name "returnTo"; _value returnTo ]
             div [ _class "form-floating pb-3" ] [
-        textarea [
-          _id          "requestText"
+                textarea [ _id          "requestText"
                            _name        "requestText"
                            _class       "form-control"
                            _style       "min-height: 8rem;"
                            _placeholder "Enter the text of the request"
-          _autofocus;  _required
-          ] [ str req.text ]
+                           _autofocus;  _required ] [ str req.Text ]
                 label [ _for "requestText" ] [ str "Prayer Request" ]
             ]
             br []
-      match isNew with
-      | true -> ()
-      | false ->
+            if not isNew then
                 div [ _class "pb-3" ] [
                     label [] [ str "Also Mark As" ]
                     br []
                     div [ _class "form-check form-check-inline" ] [
-              input [ _type "radio"; _class "form-check-input"; _id "sU"; _name "status"; _value "Updated"; _checked ]
+                        input [ _type  "radio"
+                                _class "form-check-input"
+                                _id    "sU"
+                                _name  "status"
+                                _value "Updated"
+                                _checked ]
                         label [ _for "sU" ] [ str "Updated" ]
                     ]
                     div [ _class "form-check form-check-inline" ] [
@@ -195,55 +202,53 @@ let edit (req : JournalRequest) returnTo isNew =
                     ]
                     div [ _class "d-flex flex-row flex-wrap justify-content-center align-items-center" ] [
                         div [ _class "form-check mx-2" ] [
-              input [
-                _type    "radio"
+                            input [ _type    "radio"
                                     _class   "form-check-input"
                                     _id      "rI"
                                     _name    "recurType"
                                     _value   "Immediate"
                                     _onclick "mpj.edit.toggleRecurrence(event)"
-                match req.recurType with Immediate -> _checked | _ -> ()
-                ]
+                                    match req.Recurrence with Immediate -> _checked | _ -> () ]
                             label [ _for "rI" ] [ str "Immediately" ]
                         ]
                         div [ _class "form-check mx-2"] [
-              input [
-                _type    "radio"
+                            input [ _type    "radio"
                                     _class   "form-check-input"
                                     _id      "rO"
                                     _name    "recurType"
                                     _value   "Other"
                                     _onclick "mpj.edit.toggleRecurrence(event)"
-                match req.recurType with Immediate -> () | _ -> _checked
-                ]
+                                    match req.Recurrence with Immediate -> () | _ -> _checked ]
                             label [ _for "rO" ] [ rawText "Every…" ]
                         ]
                         div [ _class "form-floating mx-2"] [
-              input [
-                _type        "number"
+                            input [ _type        "number"
                                     _class       "form-control"
                                     _id          "recurCount"
                                     _name        "recurCount"
                                     _placeholder "0"
-                _value       (string req.recurCount)
+                                    _value       recurCount
                                     _style       "width:6rem;"
                                     _required
-                match req.recurType with Immediate -> _disabled | _ -> ()
-                ]
+                                    match req.Recurrence with Immediate -> _disabled | _ -> () ]
                             label [ _for "recurCount" ] [ str "Count" ]
                         ]
                         div [ _class "form-floating mx-2" ] [
-              select [
-                _class    "form-control"
+                            select [ _class    "form-control"
                                      _id       "recurInterval"
                                      _name     "recurInterval"
                                      _style    "width:6rem;"
                                      _required
-                match req.recurType with Immediate -> _disabled | _ -> ()
-                ] [
-                option [ _value "Hours"; match req.recurType with Hours -> _selected | _ -> () ] [ str "hours" ]
-                option [ _value "Days";  match req.recurType with Days  -> _selected | _ -> () ] [ str "days" ]
-                option [ _value "Weeks"; match req.recurType with Weeks -> _selected | _ -> () ] [ str "weeks" ]
+                                     match req.Recurrence with Immediate -> _disabled | _ -> () ] [
+                                option [ _value "Hours"; match req.Recurrence with Hours _ -> _selected | _ -> () ] [
+                                    str "hours"
+                                ]
+                                option [ _value "Days";  match req.Recurrence with Days  _ -> _selected | _ -> () ] [
+                                    str "days"
+                                ]
+                                option [ _value "Weeks"; match req.Recurrence with Weeks _ -> _selected | _ -> () ] [
+                                    str "weeks"
+                                ]
                             ]
                             label [ _form "recurInterval" ] [ str "Interval" ]
                         ]
@@ -258,9 +263,9 @@ let edit (req : JournalRequest) returnTo isNew =
     ]
 
 /// Display a list of notes for a request
-let notes now notes =
+let notes now tz notes =
     let toItem (note : Note) =
-    p [] [ small [ _class "text-muted" ] [ relativeDate note.asOf now ]; br []; str note.notes ]
+        p [] [ small [ _class "text-muted" ] [ relativeDate note.AsOf now tz ]; br []; str note.Notes ]
     [   p [ _class "text-center" ] [ strong [] [ str "Prior Notes for This Request" ] ]
         match notes with
         | [] -> p [ _class "text-center text-muted" ] [ str "There are no prior notes for this request" ]

src/MyPrayerJournal/appsettings.json

@@ -1,12 +1,2 @@
-{
-  "ConnectionStrings": {
-    "db": "Filename=./mpj.db"
-  },
-  "Kestrel": {
-    "EndPoints": {
-      "Http": {
-        "Url": "http://localhost:3000"
-      }
-    }
-  }
+{
 }

src/MyPrayerJournal/wwwroot/script/mpj.js

@@ -1,7 +1,7 @@
 "use strict"
 
 /** myPrayerJournal script */
-const mpj = {
+this.mpj = {
   /**
    * Show a message via toast
    * @param {string} message The message to show
@@ -66,6 +66,19 @@ const mpj = {
       const isDisabled = target.value === "Immediate"
       ;["recurCount", "recurInterval"].forEach(it => document.getElementById(it).disabled = isDisabled)
     }
+  },
+  /**
+   * The time zone of the current browser
+   * @type {string}
+   **/
+  timeZone: undefined,
+  /**
+   * Derive the time zone from the current browser
+   */
+  deriveTimeZone () {
+    try {
+      this.timeZone = (new Intl.DateTimeFormat()).resolvedOptions().timeZone
+    } catch (_) { }
   }
 }
 
@@ -80,3 +93,12 @@ htmx.on("htmx:afterOnLoad", function (evt) {
     document.getElementById(evt.detail.xhr.getResponseHeader("x-hide-modal") + "Dismiss").click()
   }
 })
+
+htmx.on("htmx:configRequest", function (evt) {
+  // Send the user's current time zone so that we can display local time
+  if (mpj.timeZone) {
+    evt.detail.headers["X-Time-Zone"] = mpj.timeZone
+  }
+})
+
+mpj.deriveTimeZone()