Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: PRSDM-7219 / Presidium Feedback Bar Fails With Uppercase Directory Names #20

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

julianbyte
Copy link
Contributor

@julianbyte julianbyte commented Feb 3, 2025

Description

When content directories contained uppercase letters (e.g., Uppercase-Section), the id field in searchmap.json was automatically converted to lowercase, but the articleID here in the presidium-layouts-base retained its original casing. This mismatch caused ArticleBar to fail to find and display the articles when requesting them to the API.

Issue

PRSDM-7219

Screenshots

screenshot.mp4

PR Readiness Checks

  • Your PR title conforms to conventional commits <type>: <jira-ticket-num><title>, for example: fix: PRSDM-123 issue with login with a maximum of 100 characters
  • You have performed a self-review of your changes via the GitHub UI
  • Comments were added to new code that can not explain itself (see reference 1 and reference 2)
  • New code adheres to the following quality standards:

…percase, matching searchmap.json and fixing ArticleBar loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants