.gitignore

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

README.md

@@ -6,11 +6,6 @@ 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.
 
-## Further Reading
+## Futher 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 --nologo
+dotnet publish -c Release -r linux-x64 -p:PublishSingleFile=true --self-contained false

src/Dockerfile

@@ -1,17 +0,0 @@
-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

@@ -0,0 +1,22 @@
+<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

@@ -0,0 +1,57 @@
+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

@@ -1,26 +0,0 @@
-
-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,2 +1,5 @@
+## LiteDB database file
+*.db
+
 ## Development settings
 appsettings.Development.json

src/MyPrayerJournal/Data.fs

@@ -1,202 +1,209 @@
-module MyPrayerJournal.Data
+module MyPrayerJournal.Data
 
-/// Table(!) used by myPrayerJournal
+open LiteDB
-module Table =
+open NodaTime
+open System
+open System.Threading.Tasks
 
-    /// Requests
+// fsharplint:disable MemberNames
-    [<Literal>]
+
-    let Request = "mpj.request"
+/// 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
 
 
-/// JSON serialization customizations
+/// Map domain to LiteDB
+//  It does mapping, but since we're so DU-heavy, this gives us control over the JSON representation
 [<RequireQualifiedAccess>]
-module Json =
+module Mapping =
   
-    open System.Text.Json.Serialization
+  /// 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
 
-    /// Convert a wrapped DU to/from its string representation
+  /// Map a BSON document to a history entry
-    type WrappedJsonConverter<'T>(wrap : string -> 'T, unwrap : 'T -> string) =
+  let historyFromBson (doc : BsonValue) =
-        inherit JsonConverter<'T>()
+    { asOf   = Instant.FromUnixTimeMilliseconds doc["asOf"].AsInt64
-        override _.Read(reader, _, _) =
+      status = RequestAction.ofString doc["status"].AsString
-            wrap (reader.GetString()) 
+      text   = match doc["text"].AsString with "" -> None | txt -> Some txt
-        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
       }
 
-    /// Set up the data environment
+  /// Map a note entry to BSON
-    let setUp (cfg : IConfiguration) = backgroundTask {
+  let noteToBson (note : Note) : BsonValue =
-        let builder = NpgsqlDataSourceBuilder (cfg.GetConnectionString "mpj")
+    let doc = BsonDocument ()
-        let _ = builder.UseNodaTime()
+    doc["asOf"]  <- note.asOf.ToUnixTimeMilliseconds ()
-        Configuration.useDataSource (builder.Build())
+    doc["notes"] <- note.notes
-        Configuration.useIdField "id"
+    upcast doc
-        Configuration.useSerializer
+
-            { new IDocumentSerializer with
+  /// Map a BSON document to a note entry
-                member _.Serialize<'T>(it : 'T)       = JsonSerializer.Serialize(it, Json.options)
+  let noteFromBson (doc : BsonValue) =
-                member _.Deserialize<'T>(it : string) = JsonSerializer.Deserialize<'T>(it, Json.options)
+    { asOf  = Instant.FromUnixTimeMilliseconds doc["asOf"].AsInt64
-            }
+      notes = doc["notes"].AsString
-        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
   
-/// Data access functions for requests
+  /// Map a BSON document to a request
-[<RequireQualifiedAccess>]
+  let requestFromBson (doc : BsonValue) =
-module Request =
+    { id           = RequestId.ofString doc["_id"].AsString
-    
+      enteredOn    = Instant.FromUnixTimeMilliseconds doc["enteredOn"].AsInt64
-    open NodaTime
+      userId       = UserId doc["userId"].AsString
-
+      snoozedUntil = Instant.FromUnixTimeMilliseconds doc["snoozedUntil"].AsInt64
-    /// Add a request
+      showAfter    = Instant.FromUnixTimeMilliseconds doc["showAfter"].AsInt64
-    let add req =
+      recurType    = Recurrence.ofString doc["recurType"].AsString
-        insert<Request> Table.Request req
+      recurCount   = int16 doc["recurCount"].AsInt32
-
+      history      = doc["history"].AsArray |> Seq.map historyFromBson |> List.ofSeq
-    /// Does a request exist for the given request ID and user ID?
+      notes        = doc["notes"].AsArray   |> Seq.map noteFromBson    |> List.ofSeq
-    let existsById (reqId : RequestId) (userId : UserId) =
-        Exists.byContains Table.Request {| Id = reqId; UserId = userId |}
-    
-    /// 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
+  /// Set up the mapping
-    let updateRecurrence reqId userId (recurType : Recurrence) = backgroundTask {
+  let register () = 
-        let dbId = RequestId.toString reqId
+    BsonMapper.Global.RegisterType<Request>(
-        match! existsById reqId userId with
+      Func<Request, BsonValue> requestToBson, Func<BsonValue, Request> requestFromBson)
-        | true -> do! Patch.byId Table.Request dbId {| Recurrence = recurType |}
+
-        | false -> invalidOp $"Request ID {dbId} not found"
+/// 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
   }
 
-    /// Update the show-after time for a request
+/// Add a history entry
-    let updateShowAfter reqId userId (showAfter : Instant option) = backgroundTask {
+let addHistory reqId userId hist db = backgroundTask {
-        let dbId = RequestId.toString reqId
+  match! tryFullRequestById reqId userId db with
-        match! existsById reqId userId with
+  | Some req -> do! doUpdate db { req with history = hist :: req.history }
-        | true -> do! Patch.byId Table.Request dbId {| ShowAfter = showAfter |}
+  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-        | false -> invalidOp $"Request ID {dbId} not found"
   }
 
-    /// Update the snoozed and show-after values for a request
+/// Add a note
-    let updateSnoozed reqId userId (until : Instant option) = backgroundTask {
+let addNote reqId userId note db = backgroundTask {
-        let dbId = RequestId.toString reqId
+  match! tryFullRequestById reqId userId db with
-        match! existsById reqId userId with
+  | Some req -> do! doUpdate db { req with notes = note :: req.notes }
-        | true -> do! Patch.byId Table.Request dbId {| SnoozedUntil = until; ShowAfter = until |}
+  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-        | false -> invalidOp $"Request ID {dbId} not found"
   }
 
+/// Add a request
+let addRequest (req : Request) (db : LiteDatabase) =
+  db.requests.Insert req |> ignore
 
-/// Specific manipulation of history entries
+// FIXME: make a common function here
-[<RequireQualifiedAccess>]
-module History =
 
-    /// Add a history entry
+/// Retrieve all answered requests for the given user
-    let add reqId userId hist = backgroundTask {
+let answeredRequests userId (db : LiteDatabase) = backgroundTask {
-        let dbId = RequestId.toString reqId
+  let! reqs = db.requests.Find (Query.EQ ("userId", UserId.toString userId)) |> toListAsync
-        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.ofList
+    |> Seq.map JournalRequest.ofRequestFull
-            |> Seq.map JournalRequest.ofRequestLite
+    |> Seq.filter (fun it -> it.lastStatus = Answered)
-            |> Seq.filter (fun it -> it.LastStatus = Answered)
+    |> Seq.sortByDescending (fun it -> it.asOf)
-            |> Seq.sortByDescending (_.AsOf)
     |> List.ofSeq
   }
 
-    /// Retrieve a user's current prayer journal (includes snoozed and non-immediate recurrence)
+/// Retrieve the user's current journal
-    let forUser (userId : UserId) = backgroundTask {
+let journalByUserId userId (db : LiteDatabase) = backgroundTask {
-        let! reqs =
+  let! jrnl = db.requests.Find (Query.EQ ("userId", UserId.toString userId)) |> toListAsync
-            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
+    jrnl
-            |> Seq.ofList
     |> Seq.map JournalRequest.ofRequestLite
-            |> Seq.filter (fun it -> it.LastStatus <> Answered)
+    |> Seq.filter (fun it -> it.lastStatus <> Answered)
-            |> Seq.sortBy (_.AsOf)
+    |> Seq.sortBy (fun it -> it.asOf)
     |> List.ofSeq
   }
 
-    /// Does the user's journal have any snoozed requests?
+/// Does the user have any snoozed requests?
-    let hasSnoozed userId now = backgroundTask {
+let hasSnoozed userId now (db : LiteDatabase) = backgroundTask {
-        let! jrnl = forUser userId
+  let! jrnl = journalByUserId userId db
-        return jrnl |> List.exists (fun r -> defaultArg (r.SnoozedUntil |> Option.map (fun it -> it > now)) false)
+  return jrnl |> List.exists (fun r -> r.snoozedUntil > now)
   }
 
-    let tryById reqId userId = backgroundTask {
+/// Retrieve a request by its ID and user ID (without notes and history)
-        let! req = Request.tryById reqId userId
+let tryRequestById reqId userId db = backgroundTask {
+  let! req = tryFullRequestById reqId userId db
+  return req |> Option.map (fun r -> { r with history = []; notes = [] })
+  }
+
+/// 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
   return req |> Option.map JournalRequest.ofRequestLite
   }
     
-
+/// Update the recurrence for a request
-/// Specific manipulation of note entries
+let updateRecurrence reqId userId recurType recurCount db = backgroundTask {
-[<RequireQualifiedAccess>]
+  match! tryFullRequestById reqId userId db with
-module Note =
+  | Some req -> do! doUpdate db { req with recurType = recurType; recurCount = recurCount }
-
+  | None     -> invalidOp $"{RequestId.toString reqId} not found"
-    /// 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"
   }
 
-    /// Retrieve notes for a request by the request ID
+/// Update a snoozed request
-    let byRequestId reqId userId = backgroundTask {
+let updateSnoozed reqId userId until db = backgroundTask {
-        match! Request.tryById reqId userId with Some req -> return req.Notes | None -> return []
+  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"
   }

src/MyPrayerJournal/Dates.fs

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

src/MyPrayerJournal/Domain.fs

@@ -1,84 +1,67 @@
+[<AutoOpen>]
 /// The data model for myPrayerJournal
-[<AutoOpen>]
 module MyPrayerJournal.Domain
 
-open System
+// fsharplint:disable RecordFieldNames
+
 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
-    /// A request should reappear in the given number of hours
+  | Days
-    | Hours of int16
+  | Weeks
-    
-    /// 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   h -> $"{h} Hours"
+    | Hours     -> "Hours"
-        | Days    d -> $"{d} Days"
+    | Days      -> "Days"
-        | Weeks   w -> $"{w} Weeks"
+    | Weeks     -> "Weeks"
-    
   /// Create a recurrence value from a string
   let ofString =
     function
     | "Immediate" -> Immediate
-        | it when it.Contains " " ->
+    | "Hours"     -> Hours
-            let parts = it.Split " "
+    | "Days"      -> Days
-            let length = Convert.ToInt16 parts[0]
+    | "Weeks"     -> Weeks
-            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 =
+  let duration x =
-        function
+    (match x with
     | Immediate -> 0L
-        | Hours   h -> int64 h * oneHour
+    | Hours     -> oneHour
-        | Days    d -> int64 d * oneHour * 24L
+    | Days      -> oneHour * 24L
-        | Weeks   w -> int64 w * oneHour * 24L * 7L
+    | Weeks     -> oneHour * 24L * 7L)
 
 
 /// The action taken on a request as part of a history entry
@@ -88,9 +71,125 @@ 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
@@ -98,7 +197,6 @@ module RequestAction =
     | Prayed   -> "Prayed"
     | Updated  -> "Updated"
     | Answered -> "Answered"
-    
   /// Create a RequestAction from a string
   let ofString =
     function
@@ -107,173 +205,9 @@ 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,111 +1,104 @@
-/// 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 -> task {
+  let logOn url : HttpHandler =
+    fun next ctx -> backgroundTask {
       match url with
       | Some it ->
-            do! ctx.ChallengeAsync("Auth0", AuthenticationProperties(RedirectUri = it))
+          do! ctx.ChallengeAsync ("Auth0", AuthenticationProperties (RedirectUri = it))
           return! next ctx
       | 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.")
+    log.LogError (EventId(), ex, "An unhandled exception has occurred while executing the request.")
     clearResponse
     >=> setStatusCode 500
-        >=> setHttpHeader "X-Toast" $"error|||{ex.GetType().Name}: {ex.Message}"
+    >=> setHttpHeader "X-Toast" (sprintf "error|||%s: %s" (ex.GetType().Name) ex.Message)
     >=> text ex.Message
 
-    /// Handle unauthorized actions, redirecting to log on for GETs, otherwise returning a 401 Not Authorized response
+  /// Handle unauthorized actions, redirecting to log on for GETs, otherwise returning a 401 Not Authorized reponse
-    let notAuthorized : HttpHandler = fun next ctx ->
+  let notAuthorized : HttpHandler =
-        (if ctx.Request.Method = "GET" then logOn None next else setStatusCode 401 earlyReturn) ctx
+    fun next ctx ->
+      (next, ctx)
+      ||> match ctx.Request.Method with
+          | "GET" -> logOn None
+          | _ -> setStatusCode 401 >=> fun _ _ -> Task.FromResult<HttpContext option> None
 
   /// 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 ->
+  let createdAt url : HttpHandler =
-        Successful.CREATED
+    fun next ctx ->
-            ($"{ctx.Request.Scheme}://{ctx.Request.Host.Value}{url}" |> setHttpHeader HeaderNames.Location) next ctx
+      (sprintf "%s://%s%s" ctx.Request.Scheme ctx.Request.Host.Value url |> setHttpHeader HeaderNames.Location
+       >=> created) next ctx
   
   /// Return a 303 SEE OTHER response (forces a GET on the redirected URL)
   let seeOther (url : string) =
@@ -114,50 +107,50 @@ module private Helpers =
   /// Render a component result
   let renderComponent nodes : HttpHandler =
     noResponseCaching
-        >=> fun _ ctx -> backgroundTask {
+    >=> fun next ctx -> backgroundTask {
-            return! ctx.WriteHtmlStringAsync(ViewEngine.RenderView.AsString.htmlNodes nodes)
+      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 =
+    let! hasSnoozed = backgroundTask {
-            match ctx.CurrentUser with
+      match user ctx with
-            | Some _ -> Journal.hasSnoozed ctx.UserId (ctx.Now())
+      | Some _ -> return! Data.hasSnoozed (userId ctx) (now ctx) (db ctx)
-            | None   -> Task.FromResult false
+      | None   -> return  false
-        return
+      }
-            {   IsAuthenticated = Option.isSome ctx.CurrentUser
+    return {
-                HasSnoozed      = hasSnoozed
+      isAuthenticated = (user >> Option.isSome) ctx
-                CurrentUrl      = ctx.Request.Path.Value
+      hasSnoozed      = hasSnoozed
-                PageTitle       = pageTitle
+      currentUrl      = ctx.Request.Path.Value
-                Content         = content
+      pageTitle       = pageTitle
+      content         = content
       }
     }
 
   /// Composable handler to write a view to the output
-    let writeView view : HttpHandler = fun _ ctx -> backgroundTask {
+  let writeView view : HttpHandler =
+    fun next ctx -> backgroundTask {
       return! ctx.WriteHtmlViewAsync view
       }
 
-    
   /// Hold messages across redirects
   module Messages =
 
     /// The messages being held
-        let mutable private messages : Map<UserId, string * string> = Map.empty
+    let mutable private messages : Map<string, (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 : HttpContext) message url = lock upd8 (fun () ->
+    let push ctx message url = lock upd8 (fun () ->
-            messages <- messages.Add(ctx.UserId, (message, url)))
+      messages <- messages.Add (ctx |> (user >> Option.get), (message, url)))
 
     /// Add a success message header to the response
     let pushSuccess ctx message url =
-            push ctx $"success|||%s{message}" url
+      push ctx (sprintf "success|||%s" message) url
     
     /// Pop the messages for the given user
     let pop userId = lock upd8 (fun () ->
@@ -166,16 +159,17 @@ 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 -> task {
+  let partialStatic (pageTitle : string) content : HttpHandler =
+    fun next ctx -> backgroundTask {
       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 ctx.CurrentUser with
+        ||> match user ctx with
-                | Some _ ->
+            | Some u ->
-                    match Messages.pop ctx.UserId with
+                match Messages.pop u with
-                    | Some (msg, url) -> setHttpHeader "X-Toast" msg >=> withHxPushUrl url >=> writeView view
+                | Some (msg, url) -> setHttpHeader "X-Toast" msg >=> withHxPush url >=> writeView view
                 | None -> writeView view
             | None -> writeView view
       }
@@ -198,109 +192,104 @@ module Models =
   
   /// An additional note
   [<CLIMutable; NoComparison; NoEquality>]
-    type NoteEntry =
+  type NoteEntry = {
-        {   /// The notes being added
+    /// The notes being added
     notes : string
     }
   
   /// A prayer request
   [<CLIMutable; NoComparison; NoEquality>]
-    type Request =
+  type Request = {
-        {   /// The ID of the 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 =
+  type SnoozeUntil = {
-        {   /// The date (YYYY-MM-DD) at which the request should reappear
+    /// 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 = requireUser >=> fun next ctx -> task {
+  let journalItems : HttpHandler =
-        let now = ctx.Now ()
+    requiresAuthentication Error.notAuthorized
-        let shouldBeShown (req : JournalRequest) =
+    >=> fun next ctx -> backgroundTask {
-            match req.SnoozedUntil, req.ShowAfter with
+      let  now   = now ctx
-            | None, None -> true
+      let! jrnl  = Data.journalByUserId (userId ctx) (db ctx)
-            | Some snooze, Some hide when snooze < now && hide < now -> true
+      let  shown = jrnl |> List.filter (fun it -> now > it.snoozedUntil && now > it.showAfter)
-            | Some snooze, _ when snooze < now -> true
+      return! renderComponent [ Views.Journal.journalItems now shown ] next ctx
-            | _, 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 = requireUser >=> fun next ctx -> task {
+  let requestItem reqId : HttpHandler =
-        match! Journal.tryById (RequestId.ofString reqId) ctx.UserId with
+    requiresAuthentication Error.notAuthorized
-        | Some req -> return! renderComponent [ Views.Request.reqListItem (ctx.Now()) ctx.TimeZone req ] next ctx
+    >=> 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
       | None     -> return! Error.notFound next ctx
       }
 
   // GET /components/request/[req-id]/add-notes
   let addNotes requestId : HttpHandler =
-        requireUser >=> renderComponent (Views.Journal.notesEdit (RequestId.ofString requestId))
+    requiresAuthentication Error.notAuthorized
+    >=> renderComponent (Views.Journal.notesEdit (RequestId.ofString requestId))
 
   // GET /components/request/[req-id]/notes
-    let notes requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let notes requestId : HttpHandler =
-        let! notes = Note.byRequestId (RequestId.ofString requestId) ctx.UserId
+    requiresAuthentication Error.notAuthorized
-        return! renderComponent (Views.Request.notes (ctx.Now()) ctx.TimeZone notes) next ctx
+    >=> 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
       }
   
   // GET /components/request/[req-id]/snooze
   let snooze requestId : HttpHandler =
-        requireUser >=> renderComponent [ RequestId.ofString requestId |> Views.Journal.snooze ]
+    requiresAuthentication Error.notAuthorized
+    >=> renderComponent [ RequestId.ofString requestId |> Views.Journal.snooze ]
 
 
-/// / URL and documentation
+/// / URL    
 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 = requireUser >=> fun next ctx -> task {
+  let journal : HttpHandler =
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
       let usr =
         ctx.User.Claims
         |> Seq.tryFind (fun c -> c.Type = ClaimTypes.GivenName)
-            |> Option.map (_.Value)
+        |> Option.map (fun c -> c.Value)
         |> Option.defaultValue "Your"
-        let title = usr |> match usr with "Your" -> sprintf "%s" | _ -> sprintf "%s&rsquo;s"
+      let title = usr |> match usr with "Your" -> sprintf "%s" | _ -> sprintf "%s's"
-        return! partial $"{title} Prayer Journal" (Views.Journal.journal usr) next ctx
+      return! partial (sprintf "%s Prayer Journal" title) (Views.Journal.journal usr) next ctx
     }
 
 
@@ -319,12 +308,12 @@ module Legal =
 /// /api/request and /request(s) URLs
 module Request =
 
-    open Cuid
-    
   // GET /request/[req-id]/edit  
-    let edit requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let edit requestId : HttpHandler =
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
       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"
@@ -333,7 +322,7 @@ module Request =
           return! partial "Add Prayer Request"
                     (Views.Request.edit (JournalRequest.ofRequestLite Request.empty) returnTo true) next ctx
       | _     ->
-            match! Journal.tryById (RequestId.ofString requestId) ctx.UserId with
+          match! Data.tryJournalById (RequestId.ofString requestId) (userId ctx) (db ctx) with
           | Some req ->
               debug ctx "Found - sending view"
               return! partial "Edit Prayer Request" (Views.Request.edit req returnTo false) next ctx
@@ -343,153 +332,197 @@ module Request =
       }
 
   // PATCH /request/[req-id]/prayed
-    let prayed requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let prayed requestId : HttpHandler =
-        let userId = ctx.UserId
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
+      let db    = db     ctx
+      let usrId = userId ctx
       let reqId = RequestId.ofString requestId
-        match! Journal.tryById reqId userId with
+      match! Data.tryRequestById reqId usrId db with
       | Some req ->
-            let now  = ctx.Now ()
+          let now  = now ctx
-            do! History.add reqId userId { AsOf = now; Status = Prayed; Text = None }
+          do! Data.addHistory reqId usrId { asOf = now; status = Prayed; text = None } db
           let nextShow =
-                match Recurrence.duration req.Recurrence with
+            match Recurrence.duration req.recurType with
-                | 0L       -> None
+            | 0L       -> Instant.MinValue
-                | duration -> Some <| now.Plus (Duration.FromSeconds duration)
+            | duration -> now.Plus (Duration.FromSeconds (duration * int64 req.recurCount))
-            do! Request.updateShowAfter reqId userId nextShow
+          do! Data.updateShowAfter reqId usrId nextShow db
+          do! db.saveChanges ()
           return! (withSuccessMessage "Request marked as prayed" >=> Components.journalItems) next ctx
       | None -> return! Error.notFound next ctx
       }
   
-    // POST /request/[req-id]/note
+  /// POST /request/[req-id]/note
-    let addNote requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let addNote requestId : HttpHandler =
-        let userId = ctx.UserId
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
+      let db    = db     ctx
+      let usrId = userId ctx
       let reqId = RequestId.ofString requestId
-        match! Request.existsById reqId userId with
+      match! Data.tryRequestById reqId usrId db with
-        | true ->
+      | Some _ ->
-            let! notes = ctx.BindFormAsync<Models.NoteEntry>()
+          let! notes = ctx.BindFormAsync<Models.NoteEntry> ()
-            do! Note.add reqId userId { AsOf = ctx.Now(); Notes = notes.notes }
+          do! Data.addNote reqId usrId { asOf = now ctx; notes = notes.notes } db
+          do! db.saveChanges ()
           return! (withSuccessMessage "Added Notes" >=> hideModal "notes" >=> created) next ctx
-        | false -> return! Error.notFound next ctx
+      | None -> return! Error.notFound next ctx
       }
           
   // GET /requests/active
-    let active : HttpHandler = requireUser >=> fun next ctx -> task {
+  let active : HttpHandler =
-        let! reqs = Journal.forUser ctx.UserId
+    requiresAuthentication Error.notAuthorized
-        return! partial "Active Requests" (Views.Request.active (ctx.Now()) ctx.TimeZone reqs) next ctx
+    >=> fun next ctx -> backgroundTask {
+      let! reqs = Data.journalByUserId (userId ctx) (db ctx)
+      return! partial "Active Requests" (Views.Request.active (now ctx) reqs) next ctx
       }
   
   // GET /requests/snoozed
-    let snoozed : HttpHandler = requireUser >=> fun next ctx -> task {
+  let snoozed : HttpHandler =
-        let! reqs    = Journal.forUser ctx.UserId
+    requiresAuthentication Error.notAuthorized
-        let  now     = ctx.Now()
+    >=> fun next ctx -> backgroundTask {
-        let  snoozed = reqs
+      let! reqs    = Data.journalByUserId (userId ctx) (db ctx)
-                       |> List.filter (fun it -> defaultArg (it.SnoozedUntil |> Option.map (fun it -> it > now)) false)
+      let  now     = now ctx
-        return! partial "Snoozed Requests" (Views.Request.snoozed now ctx.TimeZone snoozed) next ctx
+      let  snoozed = reqs |> List.filter (fun it -> it.snoozedUntil > now)
+      return! partial "Active Requests" (Views.Request.snoozed now snoozed) next ctx
       }
 
   // GET /requests/answered
-    let answered : HttpHandler = requireUser >=> fun next ctx -> task {
+  let answered : HttpHandler =
-        let! reqs = Journal.answered ctx.UserId
+    requiresAuthentication Error.notAuthorized
-        return! partial "Answered Requests" (Views.Request.answered (ctx.Now()) ctx.TimeZone reqs) next ctx
+    >=> 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
       }
   
   // GET /request/[req-id]/full
-    let getFull requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let getFull requestId : HttpHandler =
-        match! Request.tryById (RequestId.ofString requestId) ctx.UserId with
+    requiresAuthentication Error.notAuthorized
-        | Some req -> return! partial "Prayer Request" (Views.Request.full ctx.Clock ctx.TimeZone req) next ctx
+    >=> 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
       | None     -> return! Error.notFound next ctx
       }
   
   // PATCH /request/[req-id]/show
-    let show requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let show requestId : HttpHandler =
-        let userId = ctx.UserId
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
+      let db    = db     ctx
+      let usrId = userId ctx
       let reqId = RequestId.ofString requestId
-        match! Request.existsById reqId userId with
+      match! Data.tryRequestById reqId usrId db with
-        | true ->
+      | Some _ ->
-            do! Request.updateShowAfter reqId userId None
+          do! Data.updateShowAfter reqId usrId Instant.MinValue db
+          do! db.saveChanges ()
           return! (withSuccessMessage "Request now shown" >=> Components.requestItem requestId) next ctx
-        | false -> return! Error.notFound next ctx
+      | None -> return! Error.notFound next ctx
       }
   
   // PATCH /request/[req-id]/snooze
-    let snooze requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let snooze requestId : HttpHandler =
-        let userId = ctx.UserId
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
+      let db    = db     ctx
+      let usrId = userId ctx
       let reqId = RequestId.ofString requestId
-        match! Request.existsById reqId userId with
+      match! Data.tryRequestById reqId usrId db with
-        | true ->
+      | Some _ ->
-            let! until = ctx.BindFormAsync<Models.SnoozeUntil>()
+          let! until = ctx.BindFormAsync<Models.SnoozeUntil> ()
           let date =
             LocalDatePattern.CreateWithInvariantCulture("yyyy-MM-dd").Parse(until.until).Value
               .AtStartOfDayInZone(DateTimeZone.Utc)
-                    .ToInstant()
+              .ToInstant ()
-            do! Request.updateSnoozed reqId userId (Some date)
+          do! Data.updateSnoozed reqId usrId date db
+          do! db.saveChanges ()
           return!
             (withSuccessMessage $"Request snoozed until {until.until}"
              >=> hideModal "snooze"
              >=> Components.journalItems) next ctx
-        | false -> return! Error.notFound next ctx
+      | None -> return! Error.notFound next ctx
       }
   
   // PATCH /request/[req-id]/cancel-snooze
-    let cancelSnooze requestId : HttpHandler = requireUser >=> fun next ctx -> task {
+  let cancelSnooze requestId : HttpHandler =
-        let userId = ctx.UserId
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> backgroundTask {
+      let db    = db     ctx
+      let usrId = userId ctx
       let reqId = RequestId.ofString requestId
-        match! Request.existsById reqId userId with
+      match! Data.tryRequestById reqId usrId db with
-        | true ->
+      | Some _ ->
-            do! Request.updateSnoozed reqId userId None
+          do! Data.updateSnoozed reqId usrId Instant.MinValue db
+          do! db.saveChanges ()
           return! (withSuccessMessage "Request unsnoozed" >=> Components.requestItem requestId) next ctx
-        | false -> return! Error.notFound next ctx
+      | None -> return! Error.notFound next ctx
       }
 
-    /// Derive a recurrence from its representation in the form
+  /// Derive a recurrence and interval from its primitive representation in the form
   let private parseRecurrence (form : Models.Request) =
-        match form.recurInterval with Some x -> $"{defaultArg form.recurCount 0s} {x}" | None -> "Immediate"
+    (Recurrence.ofString (match form.recurInterval with Some x -> x | _ -> "Immediate"),
-        |> Recurrence.ofString
+     defaultArg form.recurCount (int16 0))
 
   // POST /request
-    let add : HttpHandler = requireUser >=> fun next ctx -> task {
+  let add : HttpHandler =
-        let! form   = ctx.BindModelAsync<Models.Request>()
+    requiresAuthentication Error.notAuthorized
-        let  userId = ctx.UserId
+    >=> fun next ctx -> backgroundTask {
-        let  now    = ctx.Now()
+      let! form             = ctx.BindModelAsync<Models.Request> ()
+      let  db               = db ctx
+      let  usrId            = userId ctx
+      let  now              = now ctx
+      let (recur, interval) = parseRecurrence form
       let  req   =
         { Request.empty with
-                Id         = Cuid.generate () |> RequestId
+            userId     = usrId
-                UserId     = userId
+            enteredOn  = now
-                EnteredOn  = now
+            showAfter  = Instant.MinValue
-                ShowAfter  = None
+            recurType  = recur
-                Recurrence = parseRecurrence form
+            recurCount = interval
-                History    = [
+            history    = [
-                    {   AsOf   = now
+              { asOf   = now
-                        Status = Created
+                status = Created
-                        Text   = Some form.requestText
+                text   = Some form.requestText
                 }      
               ]
           }
-        do! Request.add req
+      Data.addRequest req db
+      do! db.saveChanges ()
       Messages.pushSuccess ctx "Added prayer request" "/journal"
       return! seeOther "/journal" next ctx
       }
   
   // PATCH /request
-    let update : HttpHandler = requireUser >=> fun next ctx -> task {
+  let update : HttpHandler =
-        let! form   = ctx.BindModelAsync<Models.Request>()
+    requiresAuthentication Error.notAuthorized
-        let  userId = ctx.UserId
+    >=> fun next ctx -> backgroundTask {
-        // TODO: update the instance and save rather than all these little updates
+      let! form  = ctx.BindModelAsync<Models.Request> ()
-        match! Journal.tryById (RequestId.ofString form.requestId) userId with
+      let  db    = db ctx
+      let  usrId = userId ctx
+      match! Data.tryJournalById (RequestId.ofString form.requestId) usrId db with
       | Some req ->
           // update recurrence if changed
-            let recur = parseRecurrence form
+          let (recur, interval) = parseRecurrence form
-            match recur = req.Recurrence with
+          match recur = req.recurType && interval = req.recurCount with
           | true  -> ()
           | false ->
-                do! Request.updateRecurrence req.RequestId userId recur
+              do! Data.updateRecurrence req.requestId usrId recur interval db
               match recur with
-                | Immediate -> do! Request.updateShowAfter req.RequestId userId None
+              | Immediate -> do! Data.updateShowAfter req.requestId usrId Instant.MinValue db
               | _         -> ()
           // append history
-            let upd8Text = form.requestText.Trim()
+          let upd8Text = form.requestText.Trim ()
-            let text     = if upd8Text = req.Text then None else Some upd8Text
+          let text     = match upd8Text = req.text with true -> None | false -> Some upd8Text
-            do! History.add req.RequestId userId
+          do! Data.addHistory req.requestId usrId
-                    { AsOf = ctx.Now(); Status = (Option.get >> RequestAction.ofString) form.status; Text = text }
+                { asOf = now ctx; status = (Option.get >> RequestAction.ofString) form.status; text = text } db
+          do! db.saveChanges ()
           let nextUrl =
             match form.returnTo with
             | "active"          -> "/requests/active"
@@ -504,7 +537,6 @@ module Request =
 /// Handlers for /user URLs
 module User =
 
-    open Microsoft.AspNetCore.Authentication
   open Microsoft.AspNetCore.Authentication.Cookies
 
   // GET /user/log-on
@@ -512,8 +544,10 @@ module User =
     logOn (Some "/journal")
   
   // GET /user/log-off
-    let logOff : HttpHandler = requireUser >=> fun next ctx -> task {
+  let logOff : HttpHandler =
-        do! ctx.SignOutAsync("Auth0", AuthenticationProperties (RedirectUri = "/"))
+    requiresAuthentication Error.notAuthorized
+    >=> fun next ctx -> task {
+      do! ctx.SignOutAsync ("Auth0", AuthenticationProperties (RedirectUri = "/"))
       do! ctx.SignOutAsync CookieAuthenticationDefaults.AuthenticationScheme
       return! next ctx
     }
@@ -522,8 +556,8 @@ module User =
 open Giraffe.EndpointRouting
 
 /// The routes for myPrayerJournal
-let routes = [
+let routes =
-    GET_HEAD [ route "/" Home.home ]
+  [ GET_HEAD [ route "/" Home.home ]
     subRoute "/components/" [
       GET_HEAD [
         route  "journal-items"        Components.journalItems
@@ -533,7 +567,6 @@ let routes = [
         routef "request/%s/snooze"    Components.snooze
         ]
       ]
-    GET_HEAD [ route "/docs"    Home.docs ]
     GET_HEAD [ route "/journal" Journal.journal ]
     subRoute "/legal/" [
       GET_HEAD [
@@ -567,4 +600,4 @@ let routes = [
         route "log-on"  User.logOn
         ]
       ]
-]
+    ]

src/MyPrayerJournal/MyPrayerJournal.fsproj

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

src/MyPrayerJournal/Program.fs

@@ -1,65 +1,105 @@
 module MyPrayerJournal.Api
 
-open Microsoft.AspNetCore.Http
+open Microsoft.AspNetCore.Builder
+open Microsoft.AspNetCore.Hosting
+open System.IO
 
-let sameSite (opts : CookieOptions) =
+/// 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
       | _, _ -> ()
 
-open Giraffe
+    bldr.Services
-open Giraffe.EndpointRouting
+      .AddRouting()
-open Microsoft.AspNetCore.Authentication.Cookies
+      .AddGiraffe()
-open Microsoft.AspNetCore.Authentication.OpenIdConnect
+      .AddSingleton<IClock>(SystemClock.Instance)
-open Microsoft.AspNetCore.Builder
+      .Configure<CookiePolicyOptions>(
-open Microsoft.AspNetCore.HttpOverrides
+        fun (opts : CookiePolicyOptions) ->
-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)
-    let _ =
+      .AddAuthentication(
-        svc.AddAuthentication(fun opts ->
+        /// Use HTTP "Bearer" authentication with JWTs
+        fun opts ->
           opts.DefaultAuthenticateScheme <- CookieAuthenticationDefaults.AuthenticationScheme
           opts.DefaultSignInScheme       <- CookieAuthenticationDefaults.AuthenticationScheme
           opts.DefaultChallengeScheme    <- CookieAuthenticationDefaults.AuthenticationScheme)
       .AddCookie()
-            .AddOpenIdConnect("Auth0", fun opts ->
+      .AddOpenIdConnect("Auth0",
-                // Configure OIDC with Auth0 options from configuration
+        /// Configure OIDC with Auth0 options from configuration
-                let auth0 = cfg.GetSection "Auth0"
+        fun opts ->
-                opts.Authority    <- $"""https://{auth0["Domain"]}/"""
+          let cfg = bldr.Configuration.GetSection "Auth0"
-                opts.ClientId     <- auth0["Id"]
+          opts.Authority    <- sprintf "https://%s/" cfg["Domain"]
-                opts.ClientSecret <- auth0["Secret"]
+          opts.ClientId     <- cfg["Id"]
+          opts.ClientSecret <- cfg["Secret"]
           opts.ResponseType <- OpenIdConnectResponseType.Code
           
-                opts.Scope.Clear()
+          opts.Scope.Clear ()
           opts.Scope.Add "openid"
           opts.Scope.Add "profile"
 
@@ -67,7 +107,7 @@ let main args =
           opts.ClaimsIssuer <- "Auth0"
           opts.SaveTokens   <- true
 
-                opts.Events <- OpenIdConnectEvents()
+          opts.Events <- OpenIdConnectEvents ()
           opts.Events.OnRedirectToIdentityProviderForSignOut <- fun ctx ->
             let returnTo =
               match ctx.Properties.RedirectUri with
@@ -78,34 +118,64 @@ let main args =
                     | true ->
                         // transform to absolute
                         let request = ctx.Request
-                                    $"{request.Scheme}://{request.Host.Value}{request.PathBase.Value}{redirUri}"
+                        sprintf "%s://%s%s%s" request.Scheme request.Host.Value request.PathBase.Value redirUri
                     | false -> redirUri
-                            Uri.EscapeDataString $"&returnTo={finalRedirUri}"
+                  Uri.EscapeDataString finalRedirUri |> sprintf "&returnTo=%s"
-                    ctx.Response.Redirect $"""https://{auth0["Domain"]}/v2/logout?client_id={auth0["Id"]}{returnTo}"""
+            sprintf "https://%s/v2/logout?client_id=%s%s" cfg["Domain"] cfg["Id"] returnTo
-                    ctx.HandleResponse()
+            |> ctx.Response.Redirect
+            ctx.HandleResponse ()
+
             Task.CompletedTask
           opts.Events.OnRedirectToIdentityProvider <- fun ctx ->
-                    let uri = UriBuilder ctx.ProtocolMessage.RedirectUri
+            let bldr = UriBuilder ctx.ProtocolMessage.RedirectUri
-                    uri.Scheme <- auth0["Scheme"]
+            bldr.Scheme <- cfg["Scheme"]
-                    uri.Port   <- int auth0["Port"]
+            bldr.Port   <- int cfg["Port"]
-                    ctx.ProtocolMessage.RedirectUri <- string uri
+            ctx.ProtocolMessage.RedirectUri <- string bldr
-                    Task.CompletedTask)
+            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 _ = svc.AddSingleton<JsonSerializerOptions> Json.options
-    let _ = svc.AddSingleton<Json.ISerializer>(SystemTextJson.Serializer Json.options)
-    let _ = Connection.setUp cfg |> Async.AwaitTask |> Async.RunSynchronously
 
-    if builder.Environment.IsDevelopment() then builder.Logging.AddFilter(fun l -> l > LogLevel.Information) |> ignore
+  open Giraffe.EndpointRouting
-    let _ = builder.Logging.AddConsole().AddDebug() |> ignore
 
-    use app = builder.Build()
+  /// Configure the web application
-    let _ = app.UseStaticFiles()
+  let application (app : WebApplication) =
-    let _ = app.UseCookiePolicy()
+    // match app.Environment.IsDevelopment () with
-    let _ = app.UseRouting()
+    // | true -> app.UseDeveloperExceptionPage ()
-    let _ = app.UseAuthentication()
+    // | false -> app.UseGiraffeErrorHandler Handlers.Error.error
-    let _ = app.UseGiraffeErrorHandler Handlers.Error.error
+    // |> ignore
-    let _ = app.UseEndpoints(fun e -> e.MapGiraffeEndpoints Handlers.routes)
+    app
+      .UseStaticFiles()
+      .UseCookiePolicy()
+      .UseRouting()
+      .UseAuthentication()
+      .UseGiraffeErrorHandler(Handlers.Error.error)
+      .UseEndpoints (fun e -> e.MapGiraffeEndpoints Handlers.routes |> ignore)
+    |> ignore
+    app
 
-    app.Run()
+  /// Compose all the configurations into one
+  let webHost pathSegments =
+    contentRoot
+    >> appConfiguration
+    >> kestrel
+    >> webRoot pathSegments
+    >> logging
+    >> services
+    >> application
 
+
+[<EntryPoint>]
+let main _ =
+  use host = Configure.webHost [| "wwwroot" |] (Directory.GetCurrentDirectory ())
+  host.Run ()
   0

src/MyPrayerJournal/Views/Docs.fs

@@ -1,184 +0,0 @@
-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,7 +2,6 @@
 [<AutoOpen>]
 module private MyPrayerJournal.Views.Helpers
   
-open Giraffe.Htmx
 open Giraffe.ViewEngine
 open Giraffe.ViewEngine.Htmx
 open MyPrayerJournal
@@ -11,7 +10,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 "true" ]
+  |> List.append [ _href href; _hxBoost; _hxTarget "#top"; _hxSwap HxSwap.InnerHtml; _hxPushUrl ]
   |> a
 
 /// Create a Material icon
@@ -28,15 +27,5 @@ 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 (tz : DateTimeZone) =
+let relativeDate (date : Instant) now =
-    span [ _title (date.InZone(tz).ToDateTimeOffset().ToString("f", null)) ] [ Dates.formatDistance now date |> str ]
+  span [ _title (date.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,66 +1,60 @@
 /// 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 tz req =
+let journalCard now req =
-    let reqId = RequestId.toString req.RequestId
+  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 ] [
+          _hxSwap   HxSwap.InnerHtml
-                    icon "comment"
+          ] [ 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 ] [
+          _hxSwap   HxSwap.InnerHtml
-                    icon "schedule"
+          ] [ 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" ] [
+          _title   "Mark as Prayed"
-                    icon "done"
+          ] [ 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 [] [
+        em [] [ str "last activity "; relativeDate req.asOf now ]
-                    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 =
+let journal user = article [ _class "container-fluid mt-3" ] [
-    article [ _class "container-fluid mt-3" ] [
   h2 [ _class "pb-3" ] [
     str user
     match user with "Your" -> () | _ -> rawText "’s"
@@ -72,11 +66,13 @@ let journal user =
   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" ] [
@@ -85,21 +81,20 @@ let journal user =
           ]
         div [ _class "modal-body"; _id "notesBody" ] [ ]
         div [ _class "modal-footer" ] [
-                        button [ _type  "button"
+          button [ _type "button"; _id "notesDismiss"; _class "btn btn-secondary"; _data "bs-dismiss" "modal" ] [
-                                 _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" ] [
@@ -108,10 +103,7 @@ let journal user =
           ]
         div [ _class "modal-body"; _id "snoozeBody" ] [ ]
         div [ _class "modal-footer" ] [
-                        button [ _type "button"
+          button [ _type "button"; _id "snoozeDismiss"; _class "btn btn-secondary"; _data "bs-dismiss" "modal" ] [
-                                 _id "snoozeDismiss"
-                                 _class "btn btn-secondary"
-                                 _data "bs-dismiss" "modal" ] [
             str "Close"
             ]
           ]
@@ -121,7 +113,7 @@ let journal user =
   ]
 
 /// The journal items
-let journalItems now tz items =
+let journalItems now items =
   match items |> List.isEmpty with
   | true ->
       noResults "No Active Requests" "/request/new/edit" "Add a Request" [
@@ -130,24 +122,27 @@ let journalItems now tz items =
         ]
   | false ->
       items
-        |> List.map (journalCard now tz)
+      |> List.map (journalCard now)
-        |> section [ _id        "journalItems"
+      |> 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" ] ]
@@ -155,13 +150,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" ] [
+          _hxTarget "#priorNotes"
-                    str "Load Prior Notes"
+          ] [str "Load Prior Notes" ]
-                ]
         ]
       ]
     ]
@@ -169,9 +164,11 @@ 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,125 +1,151 @@
 /// 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 =
+type PageRenderContext = {
-    { /// Whether the user is authenticated
+  /// Whether the user is authenticated
-      IsAuthenticated: bool
+  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 =
+let home = article [ _class "container mt-3" ] [
-    article [ _class "container mt-3" ]
+  p [] [ rawText " " ]
-        [ p [] [ rawText " " ]
+  p [] [
-          p []
+    str "myPrayerJournal is a place where individuals can record their prayer requests, record that they prayed for "
-            [ str "myPrayerJournal is a place where individuals can record their prayer requests, record that they "
+    str "them, update them as God moves in the situation, and record a final answer received on that request. It also "
-              str "prayed for them, update them as God moves in the situation, and record a final answer received on "
+    str "allows individuals to review their answered prayers."
-              str "that request. It also allows individuals to review their answered prayers." ]
+    ]
-          p []
+  p [] [
-            [ str "This site is open and available to the general public. To get started, simply click the "
+    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 "“Log On” link above, and log on with either a Microsoft or Google account. You can also "
-              rawText "also learn more about the site at the “Docs” link, also above." ] ]
+    rawText "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 ]
+  nav [ _class "navbar navbar-dark"; _roleNavigation ] [
-        [ div [ _class "container-fluid" ]
+    div [ _class "container-fluid" ] [
-            [ pageLink
+      pageLink "/" [ _class "navbar-brand" ] [
-                "/" [ _class "navbar-brand" ]
+        span [ _class "m" ] [ str "my" ]
-                [ 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
-                if ctx.IsAuthenticated then
+        match ctx.isAuthenticated with
+        | true ->
             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" ] ]
-                else li [ _class "nav-item"] [ a [ _href "/user/log-on" ] [ str "Log On" ] ]
+        | false -> li [ _class "nav-item"] [ a [ _href "/user/log-on" ] [ str "Log On" ] ]
-                li [ _class "nav-item" ] [ navLink "/docs" [ str "Docs" ] ]
+        li [ _class "nav-item" ] [
+          a [ _href "https://docs.prayerjournal.me"; _target "_blank"; _rel "noopener" ] [ 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 =
+let titleTag ctx = title [] [ str ctx.pageTitle; rawText " « myPrayerJournal" ]
-    title [] [ rawText ctx.PageTitle; rawText " « myPrayerJournal" ]
 
 /// The HTML `head` element
 let htmlHead ctx =
-    head [ _lang "en" ]
+  head [ _lang "en" ] [
-        [ meta [ _name "viewport"; _content "width=device-width, initial-scale=1" ]
+    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.3.2/dist/css/bootstrap.min.css"
+    link [
+      _href        "https://cdn.jsdelivr.net/npm/bootstrap@5.0.2/dist/css/bootstrap.min.css"
       _rel         "stylesheet"
-                 _integrity   "sha384-T3c6CoIi6uLrA9TneNEoa7RxnatzjcDSCmG1MXxSR1GAsXEV/Dwwykc2MPK8M2HN"
+      _integrity   "sha384-EVSTQN3/azprG1Anm3QDgpJLIm9Nao0Yz1ztcQTwFspd3yD65VohhpuuCOmLASjC"
-                 _crossorigin "anonymous" ]
+      _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 [ _ariaLive "polite"; _ariaAtomic "true"; _id "toastHost" ] [
-        [ div [ _class "toast-container position-absolute p-3 bottom-0 end-0"; _id "toasts" ] [] ]
+    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" ]
+  footer [ _class "container-fluid" ] [
-        [ p [ _class "text-muted text-end" ]
+    p [ _class "text-muted text-end" ] [
-            [ str $"myPrayerJournal {version}"
+      str "myPrayerJournal v3"
       br []
-              em []
+      em [] [
-                  [ small []
+        small [] [
-                        [ pageLink "/legal/privacy-policy" [] [ str "Privacy Policy" ]
+          pageLink "/legal/privacy-policy" [] [ str "Privacy Policy" ]
           rawText " • "
           pageLink "/legal/terms-of-service" [] [ str "Terms of Service" ]
           rawText " • "
-                          a [ _href "https://git.bitbadger.solutions/bit-badger/myPrayerJournal"
+          a [ _href "https://github.com/bit-badger/myprayerjournal"; _target "_blank"; _rel "noopener" ] [
-                              _target "_blank"
+            str "Developed"
-                              _rel "noopener" ] [ str "Developed" ]
+            ]
           str " and hosted by "
-                          a   [ _href "https://bitbadger.solutions"; _target "_blank"; _rel "noopener" ]
+          a [ _href "https://bitbadger.solutions"; _target "_blank"; _rel "noopener" ] [ str "Bit Badger Solutions" ]
-                              [ str "Bit Badger Solutions" ] ] ] ]
+          ]
-          Htmx.Script.minified
+        ]
-          script [] [ rawText "if (!htmx) document.write('<script src=\"/script/htmx.min.js\"><\/script>')" ]
+      ]
-          script [ _async
+    script [
-                   _src         "https://cdn.jsdelivr.net/npm/bootstrap@5.3.2/dist/js/bootstrap.bundle.min.js"
+      _src         "https://unpkg.com/htmx.org@1.5.0"
-                   _integrity   "sha384-C6RzsynM9kWDrMNeT87bh95OGNyZPhcTNXj1NW7RuBCsyN/o0jlpcV8Qyq46cDfL"
+      _integrity   "sha384-oGA+prIp5Vchu6we2YkI51UtVzN9Jpx2Z7PnR1I78PnZlN8LkrCT4lqqqmDkyrvI"
-                   _crossorigin "anonymous" ] []
+      _crossorigin "anonymous"
-          script []
+      ] []
-              [ rawText "setTimeout(function () { "
+    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 () { "
       rawText "if (!bootstrap) document.write('<script src=\"/script/bootstrap.bundle.min.js\"><\/script>') "
-                rawText "}, 2000)" ]
+      rawText "}, 2000)"
-          script [ _src "/script/mpj.js" ] [] ]
+      ]
+    script [ _src "/script/mpj.js" ] []
+    ]
 
 /// Create the full view of the page
 let view ctx =
-    html [ _lang "en" ]
+  html [ _lang "en" ] [
-        [ htmlHead ctx
+    htmlHead ctx
-          body []
+    body [] [
-            [ section [ _id "top"; _ariaLabel "Top navigation" ] [ navBar ctx; main [ _roleMain ] [ ctx.Content ] ]
+      section [ _id "top" ] [ 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,26 +4,23 @@ module MyPrayerJournal.Views.Legal
 open Giraffe.ViewEngine
 
 /// View for the "Privacy Policy" page
-let privacyPolicy =
+let privacyPolicy = article [ _class "container mt-3" ] [
-    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 "
+    str "The nature of the service is one where privacy is a must. The items below will help you understand the data "
-            str "the data we collect, access, and store on your behalf as you use this service."
+    str "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 "
+          str "myPrayerJournal utilizes a third-party authentication and identity provider. You should familiarize "
-                        str "familiarize yourself with the privacy policy for "
+          str "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" ] [
+          a [ _href "https://privacy.microsoft.com/en-us/privacystatement"; _target "_blank" ] [ str "Microsoft"]
-                            str "Microsoft"
-                        ]
           str " or "
           a [ _href "https://policies.google.com/privacy"; _target "_blank" ] [ str "Google" ]
           str ")."
@@ -34,23 +31,22 @@ let privacyPolicy =
         h4 [] [ str "Identifying Data" ]
         ul [] [
           li [] [
-                            str "The only identifying data myPrayerJournal stores is the subscriber "
+            rawText "The only identifying data myPrayerJournal stores is the subscriber (“sub”) field from "
-                            rawText "(“sub”) field from the token we receive from Auth0, once you have "
+            str "the token we receive from Auth0, once you have signed in through their hosted service. All "
-                            str "signed in through their hosted service. All information is associated with you via "
+            str "information is associated with you via this field."
-                            str "this field."
             ]
           li [] [
-                            str "While you are signed in, within your browser, the service has access to your first "
+            str "While you are signed in, within your browser, the service has access to your first and last names, "
-                            str "and last names, along with a URL to the profile picture (provided by your selected "
+            str "along with a URL to the profile picture (provided by your selected identity provider). This "
-                            str "identity provider). This information is not transmitted to the server, and is removed "
+            rawText "information is not transmitted to the server, and is removed when “Log Off” is "
-                            rawText "when “Log Off” is clicked."
+            str "clicked."
             ]
           ]
         h4 [] [ str "User Provided Data" ]
         ul [ _class "mb-0" ] [
           li [] [
-                            str "myPrayerJournal stores the information you provide, including the text of prayer "
+            str "myPrayerJournal stores the information you provide, including the text of prayer requests, updates, "
-                            str "requests, updates, and notes; and the date/time when certain actions are taken."
+            str "and notes; and the date/time when certain actions are taken."
             ]
           ]
         ]
@@ -58,38 +54,35 @@ let privacyPolicy =
         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 "
+            str "Your provided data is returned to you, as required, to display your journal or your answered "
-                            str "answered requests. On the server, it is stored in a controlled-access database."
+            str "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 "
+            str "Your data is backed up, along with other Bit Badger Solutions hosted systems, in a rolling manner; "
-                            str "rolling manner; backups are preserved for the prior 7 days, and backups from the 1"
+            str "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 "
+            str " are preserved for 3 months. These backups are stored in a private cloud data repository."
-                            str "repository."
             ]
           li [] [
-                            str "The data collected and stored is the absolute minimum necessary for the functionality "
+            str "The data collected and stored is the absolute minimum necessary for the functionality of the service. "
-                            rawText "of the service. There are no plans to “monetize” this service, and "
+            rawText "There are no plans to “monetize” this service, and storing the minimum amount of "
-                            str "storing the minimum amount of information means that the data we have is not "
+            str "information means that the data we have is not interesting to purchasers (or those who may have more "
-                            str "interesting to purchasers (or those who may have more nefarious purposes)."
+            str "nefarious purposes)."
             ]
           li [] [
-                            str "Access to servers and backups is strictly controlled and monitored for unauthorized "
+            str "Access to servers and backups is strictly controlled and monitored for unauthorized access attempts."
-                            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 "
+          str "At any time, you may choose to discontinue using this service. Both Microsoft and Google provide ways "
-                        str "provide ways to revoke access from this application. However, if you want your data "
+          str "to revoke access from this application. However, if you want your data removed from the database, "
-                        str "removed from the database, please contact daniel at bitbadger.solutions (via e-mail, "
+          str "please contact daniel at bitbadger.solutions (via e-mail, replacing at with @) prior to doing so, to "
-                        str "replacing at with @) prior to doing so, to ensure we can determine which subscriber ID "
+          str "ensure we can determine which subscriber ID belongs to you."
-                        str "belongs to you."
           ]
         ]
       ]
@@ -97,8 +90,7 @@ let privacyPolicy =
   ]
 
 /// View for the "Terms of Service" page
-let termsOfService =
+let termsOfService = article [ _class "container mt-3" ] [
-    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" ] [
@@ -106,17 +98,17 @@ let termsOfService =
       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, "
+          str "By accessing this web site, you are agreeing to be bound by these Terms and Conditions, and that you "
-                        str "and that you are responsible to ensure that your use of this site complies with all "
+          str "are responsible to ensure that your use of this site complies with all applicable laws. Your continued "
-                        str "applicable laws. Your continued use of this site implies your acceptance of these terms."
+          str "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 "
+          str "myPrayerJournal is a service that allows individuals to enter and amend their prayer requests. It "
-                        str "requests. It requires no registration by itself, but access is granted based on a "
+          str "requires no registration by itself, but access is granted based on a successful login with an external "
-                        str "successful login with an external identity provider. See "
+          str "identity provider. See "
           pageLink "/legal/privacy-policy" [] [ str "our privacy policy" ]
           str " for details on how that information is accessed and stored."
           ]
@@ -124,13 +116,11 @@ let termsOfService =
       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 "
+          str "This service utilizes a third-party service provider for identity management. Review the terms of "
-                        str "terms of service for "
+          str "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" ] [
+          a [ _href "https://www.microsoft.com/en-us/servicesagreement"; _target "_blank" ] [ str "Microsoft"]
-                            str "Microsoft"
-                        ]
           str " or "
           a [ _href "https://policies.google.com/terms"; _target "_blank" ] [ str "Google" ]
           str ")."
@@ -139,17 +129,17 @@ let termsOfService =
       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) "
+          rawText "This service is provided “as is”, and no warranty (express or implied) exists. The "
-                        str "exists. The service and its developers may not be held liable for any damages that may "
+          str "service and its developers may not be held liable for any damages that may arise through the use of "
-                        str "arise through the use of this service."
+          str "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 "
+          str "These terms and conditions may be updated at any time, and this service does not have the capability to "
-                        str "capability to notify users when these change. The date at the top of the page will be "
+          str "notify users when these change. The date at the top of the page will be updated when any of the text of "
-                        str "updated when any of the text of these terms is updated."
+          str "these terms is updated."
           ]
         ]
       ]
@@ -160,3 +150,4 @@ let termsOfService =
     str " to learn how we handle your data."
     ]
   ]
+

src/MyPrayerJournal/Views/Request.fs

@@ -1,109 +1,108 @@
 /// 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 tz req =
+let reqListItem now req =
-    let isFuture instant = defaultArg (instant |> Option.map (fun it -> it > now)) false
+  let reqId      = RequestId.toString req.requestId
-    let reqId      = RequestId.toString req.RequestId
+  let isAnswered = req.lastStatus = Answered
-    let isAnswered = req.LastStatus = Answered
+  let isSnoozed  = req.snoozedUntil > now
-    let isSnoozed  = isFuture req.SnoozedUntil
+  let isPending  = (not isSnoozed) && req.showAfter > now
-    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"
+  div [ _class "list-group-item px-0 d-flex flex-row align-items-start"; _hxTarget "this"; _hxSwap HxSwap.OuterHtml ] [
-          _hxTarget "this"
-          _hxSwap HxSwap.OuterHtml ] [
     pageLink $"/request/{reqId}/full" [ btnClass; _title "View Full Request" ] [ icon "description" ]
-        if not isAnswered then pageLink $"/request/{reqId}/edit" [ btnClass; _title "Edit Request" ] [ icon "edit" ]
+    match isAnswered with
-        if   isSnoozed then restoreBtn "cancel-snooze" "Cancel Snooze"
+    | true  -> ()
-        elif isPending then restoreBtn "show"          "Show Now"
+    | 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"
+    | _ -> ()
     p [ _class "request-text mb-0" ] [
-            str req.Text
+      str req.text
-            if isSnoozed || isPending || isAnswered then
+      match isSnoozed || isPending || isAnswered with
+      | true ->
           br []
           small [ _class "text-muted" ] [
-                    if   isSnoozed then   [ str "Snooze expires ";       relativeDate req.SnoozedUntil.Value now tz ]
+            match () with
-                    elif isPending then   [ str "Request appears next "; relativeDate req.ShowAfter.Value    now tz ]
+            | _ when isSnoozed   -> [ str "Snooze expires ";       relativeDate req.snoozedUntil now ]
-                    else (* isAnswered *) [ str "Answered ";             relativeDate req.AsOf               now tz ]
+            | _ when isPending   -> [ str "Request appears next "; relativeDate req.showAfter    now ]
+            | _ (* isAnswered *) -> [ str "Answered ";             relativeDate req.asOf         now ]
             |> em []
             ]
+      | false -> ()
       ]
     ]
 
 /// Create a list of requests
-let reqList now tz reqs =
+let reqList now reqs =
   reqs
-    |> List.map (reqListItem now tz)
+  |> List.map (reqListItem now)
   |> div [ _class "list-group" ]
 
 /// View for Active Requests page
-let active now tz reqs =
+let active now reqs = article [ _class "container mt-3" ] [
-    article [ _class "container mt-3" ] [
   h2 [ _class "pb-3" ] [ str "Active Requests" ]
-        if List.isEmpty reqs then
+  match reqs |> List.isEmpty with
+  | true ->
       noResults "No Active Requests" "/journal" "Return to your journal"
         [ str "Your prayer journal has no active requests" ]
-        else reqList now tz reqs
+  | false -> reqList now reqs
   ]
 
 /// View for Answered Requests page
-let answered now tz reqs =
+let answered now reqs = article [ _class "container mt-3" ] [
-    article [ _class "container mt-3" ] [
   h2 [ _class "pb-3" ] [ str "Answered Requests" ]
-        if List.isEmpty reqs then
+  match reqs |> List.isEmpty with
-            noResults "No Answered Requests" "/journal" "Return to your journal" [
+  | true ->
-                str "Your prayer journal has no answered requests; once you have marked one as "
+      noResults "No Active Requests" "/journal" "Return to your journal" [
-                rawText "“Answered”, it will appear here"
+        rawText "Your prayer journal has no answered requests; once you have marked one as “Answered”, "
+        str "it will appear here"
         ]
-        else reqList now tz reqs
+  | false -> reqList now reqs
   ]
 
 /// View for Snoozed Requests page
-let snoozed now tz reqs =
+let snoozed now reqs = article [ _class "container mt-3" ] [
-    article [ _class "container mt-3" ] [
   h2 [ _class "pb-3" ] [ str "Snoozed Requests" ]
-        reqList now tz reqs
+  reqList now reqs
   ]
 
 /// View for Full Request page
-let full (clock : IClock) tz (req : Request) =
+let full (clock : IClock) (req : Request) =
-    let now = clock.GetCurrentInstant()
+  let now = clock.GetCurrentInstant ()
   let answered =
-        req.History
+    req.history
-        |> Seq.ofList
+    |> List.filter RequestAction.isAnswered
-        |> Seq.filter History.isAnswered
+    |> List.tryHead
-        |> Seq.tryHead
+    |> Option.map (fun x -> x.asOf)
-        |> Option.map (_.AsOf)
+  let prayed = (req.history |> List.filter RequestAction.isPrayed |> List.length).ToString "N0"
-    let prayed = (req.History |> List.filter History.isPrayed |> List.length).ToString "N0"
   let daysOpen =
     let asOf = defaultArg answered now
-        ((asOf - (req.History |> List.filter History.isCreated |> List.head).AsOf).TotalDays |> int).ToString "N0"
+    ((asOf - (req.history |> List.filter RequestAction.isCreated |> List.head).asOf).TotalDays |> int).ToString "N0"
   let lastText =
-        req.History
+    req.history
-        |> Seq.ofList
+    |> List.filter (fun h -> Option.isSome h.text)
-        |> Seq.filter (fun h -> Option.isSome h.Text)
+    |> List.sortByDescending (fun h -> h.asOf)
-        |> Seq.sortByDescending (_.AsOf)
+    |> List.map (fun h -> Option.get h.text)
-        |> Seq.map (fun h -> Option.get h.Text)
+    |> List.head
-        |> 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
+      req.notes
-            |> Seq.ofList
+      |> List.map (fun n -> {| asOf = n.asOf; text = Some n.notes; status = "Notes" |})
-            |> Seq.map (fun n -> {| asOf = n.AsOf; text = Some n.Notes; status = "Notes" |})
+      |> List.append (req.history |> List.map toDisp)
-            |> Seq.append (req.History |> List.map toDisp)
+      |> List.sortByDescending (fun it -> it.asOf)
-            |> Seq.sortByDescending (_.asOf)
-            |> List.ofSeq
     // Skip the first entry for answered requests; that info is already displayed
-        match answered with Some _ -> all.Tail | None -> all
+    match answered with Some _ -> all |> List.skip 1 | None -> all
   article [ _class "container mt-3" ] [
     div [_class "card" ] [
       h5 [ _class "card-header" ] [ str "Full Prayer Request" ]
@@ -112,22 +111,21 @@ let full (clock : IClock) tz (req : Request) =
           match answered with
           | Some date ->
               str "Answered "
-                        date.ToDateTimeOffset().ToString("D", null) |> str
+              date.ToDateTimeOffset().ToString ("D", null) |> str
               str " ("
-                        relativeDate date now tz
+              relativeDate date now
               rawText ") • "
           | None -> ()
-                    rawText $"Prayed %s{prayed} times • Open %s{daysOpen} days"
+          sprintf "Prayed %s times • Open %s days" prayed daysOpen |> rawText
           ]
         p [ _class "card-text" ] [ str lastText ]
         ]
       log
-            |> List.map (fun it ->
+      |> List.map (fun it -> li [ _class "list-group-item" ] [
-                li [ _class "list-group-item" ] [
         p [ _class "m-0" ] [
           str it.status
           rawText "  "
-                        small [] [ em [] [ it.asOf.ToDateTimeOffset().ToString("D", null) |> str ] ]
+          small [] [ em [] [ it.asOf.ToDateTimeOffset().ToString ("D", null) |> str ] ]
           ]
         match it.text with
         | Some txt -> p [ _class "mt-2 mb-0" ] [ str txt ]
@@ -144,45 +142,40 @@ 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 "true"
+      _hxPushUrl
-               "/request" |> match isNew with true -> _hxPost | false -> _hxPatch ] [
+      "/request" |> match isNew with true -> _hxPost | false -> _hxPatch
-            input [ _type  "hidden"
+      ] [
+      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 []
-            if not isNew then
+      match isNew with
+      | true -> ()
+      | false ->
           div [ _class "pb-3" ] [
             label [] [ str "Also Mark As" ]
             br []
             div [ _class "form-check form-check-inline" ] [
-                        input [ _type  "radio"
+              input [ _type "radio"; _class "form-check-input"; _id "sU"; _name "status"; _value "Updated"; _checked ]
-                                _class "form-check-input"
-                                _id    "sU"
-                                _name  "status"
-                                _value "Updated"
-                                _checked ]
               label [ _for "sU" ] [ str "Updated" ]
               ]
             div [ _class "form-check form-check-inline" ] [
@@ -202,53 +195,55 @@ 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.Recurrence with Immediate -> _checked | _ -> () ]
+                match req.recurType 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.Recurrence with Immediate -> () | _ -> _checked ]
+                match req.recurType 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       recurCount
+                _value       (string req.recurCount)
                 _style       "width:6rem;"
                 _required
-                                    match req.Recurrence with Immediate -> _disabled | _ -> () ]
+                match req.recurType 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.Recurrence with Immediate -> _disabled | _ -> () ] [
+                match req.recurType with Immediate -> _disabled | _ -> ()
-                                option [ _value "Hours"; match req.Recurrence with Hours _ -> _selected | _ -> () ] [
+                ] [
-                                    str "hours"
+                option [ _value "Hours"; match req.recurType with Hours -> _selected | _ -> () ] [ str "hours" ]
-                                ]
+                option [ _value "Days";  match req.recurType with Days  -> _selected | _ -> () ] [ str "days" ]
-                                option [ _value "Days";  match req.Recurrence with Days  _ -> _selected | _ -> () ] [
+                option [ _value "Weeks"; match req.recurType with Weeks -> _selected | _ -> () ] [ str "weeks" ]
-                                    str "days"
-                                ]
-                                option [ _value "Weeks"; match req.Recurrence with Weeks _ -> _selected | _ -> () ] [
-                                    str "weeks"
-                                ]
                 ]
               label [ _form "recurInterval" ] [ str "Interval" ]
               ]
@@ -263,9 +258,9 @@ let edit (req : JournalRequest) returnTo isNew =
     ]
 
 /// Display a list of notes for a request
-let notes now tz notes =
+let notes now notes =
   let toItem (note : Note) =
-        p [] [ small [ _class "text-muted" ] [ relativeDate note.AsOf now tz ]; br []; str note.Notes ]
+    p [] [ small [ _class "text-muted" ] [ relativeDate note.asOf now ]; 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,2 +1,12 @@
-{
+{
+  "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 */
-this.mpj = {
+const mpj = {
   /**
    * Show a message via toast
    * @param {string} message The message to show
@@ -66,19 +66,6 @@ this.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 (_) { }
   }
 }
 
@@ -93,12 +80,3 @@ 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()