[DEV-2350] Add logs in guide pages #1476
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MarBert
requested review from
MarcoPonchia,
marcobottaro and
tommaso1
as code owners
🦋 Changeset detectedLatest commit: 9dd0d5a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Jira Pull Request LinkThis Pull Request refers to the following Jira issue DEV-2350 |
marcobottaro
approved these changes
Jun 12, 2025
marcobottaro
added a commit
that referenced
this pull request
Jun 26, 2025
* Use standalone * Set reactStrictMode to false * fix: use env secret * fix: parse secrets temporarily * fix: linting * make dynamic pages: webinars, guide and soltuion * use env instead of secrets to building locally * refactor guide using S3 * Fix env var management * remove sitemap * Add verbose build info * add env var logs * add log * move log * Update package json * update config * Fix config * [DEV-2132] Poc standalone improve s3 fetch guide (#1354) * Fix CICD lambda permissions (#1353) * fix: lambda permissions arn * docs: added changeset * improve fetch guide from S3 * convert all gitbook page using s3 * Fix images and solutions page * Fix after review * Add changeset --------- Co-authored-by: Christian Calabrese <[email protected]> * [DEV-2184] Poc standalone fix guide version (#1365) * Fix guide version and page title * Fix release note pages * add changeset * Add debug logs * Update logs * Use secret for bucket * update yaml * Fix fetch from strapi using secrets * Fix fetch from strapi using secrets * remove debug log * Fix build locallu * [DEV-2129] Sitemap for standalone version (#1368) * Add sitemap generation scripts for dynamic content from S3 and Strapi * Update apps/nextjs-website/scripts/createSitemapJson.ts Co-authored-by: Marco Ponchia <[email protected]> * Update apps/nextjs-website/scripts/createSitemapJson.ts Co-authored-by: Marco Ponchia <[email protected]> * Update apps/nextjs-website/scripts/createSitemapJson.ts Co-authored-by: Marco Ponchia <[email protected]> * Update apps/nextjs-website/scripts/createSitemapJson.ts Co-authored-by: Marco Ponchia <[email protected]> * Update apps/nextjs-website/scripts/createSitemapJson.ts Co-authored-by: Marco Ponchia <[email protected]> * pr comments * Refactor sitemap generation to use new environment variable for guide metadata path and add script for generating guide metadata from S3 and Strapi. * sitemap --------- Co-authored-by: t <[email protected]> Co-authored-by: Marco Ponchia <[email protected]> Co-authored-by: marcobottaro <[email protected]> * Fix sitemap * [DEV-2196] Update generate metadata functions (#1392) * add metadata generator helpers * refactor guide script * add new generate scripts and update package * update sitemap * Fix after review * Add changeset * Fix after review * [DEV-2226] Fix doc s3 parser (#1407) * Fix parseS3Doc helper * add changeset * Update shaggy-lizards-lick.md Update changeset --------- Co-authored-by: marcobottaro <[email protected]> * [DEV-2131] fix dompurify warning (#1418) * Fix dompurify warning * Add changeset --------- Co-authored-by: marcobottaro <[email protected]> * [DEV-2243] parse single guide page (#1411) * add metadata generator helpers * refactor guide script * add new generate scripts and update package * update sitemap * Fix after review * Add changeset * Add parseS3DocPage helper * Update sitemap and s3Metadata helpers * Add getGuidePage and getGuidePageProps * Update page WIP * Fix menu path and update guide version logic * Update JsonMetatadata fields * Update parseS3GuidePage * Fix api.ts * Refactor cmsApi * Update guide metadata page WIP * Add cache for metadata fetches * Fix after review * Add changeset * Fix parse error * Add package-lock.json updates --------- Co-authored-by: Marco Bottaro <[email protected]> * Add sst configuration * replaced aplifiy with open next new module * refactory condition with for static parts * module outputs and first draft stendalone deploy action * update github action permission and bug fixing * update build commands * [DEV-2311] add basic isr to next (#1450) * Add basic ISR to pages * Add ISR to guide pages and build statically first pagge for guide from CMS * Add changeset * Update apps/nextjs-website/src/app/[productSlug]/[...releaseNoteSubPathSlugs]/page.tsx Co-authored-by: marcobottaro <[email protected]> --------- Co-authored-by: marcobottaro <[email protected]> * Improve debug logs * update package json file * Add session token to client s3 credentials * add force dynamic * update deploy standalone action with permissions * deploy standalone assign aws env variables * deploy standalone setup node npm cache * Remove force dynamic * remove guide entrypoint static generation * update deploy stanalone action * deploy standalone copy assets, cache and zip all functions * update deploy lambda policy, s3 bucket to deploy code, deploy opennext lambda * update deploy standalone action * update action deploy standalone: deploy all functions * git action deploy standalone, remvoved the package part * update iam policy to deploy opennext * deploy standalone create new version and update alias * fix deploy standalone action cache step * update invoke command: it requires an outfile param * website cdn new s3 origin for standalone content * update bucket content policy and cloud front aliases * Fix metadata cache * update lambda server env variables * [DEV-2323] Fetch data using CDN (#1462) * Add env var * Update fetching data helper to fetch from cdn instead of bucket * Refactor parser of S3 doc to use new env var end helpers * Add changeset * Remove console log * terrform lock file update * fix linting error * Enable cacheInterception * open next lambda new env var: NEXT_PUBLIC_FEEDBACK_FORM_ENABLED * new strapi secret in parameter store * fix terraform template and secret parameter * [DEV-2324] Fix absolute url path transform (#1472) * WIP Add test to fix * Fix absolute url path transform --------- Co-authored-by: Marco Ponchia <[email protected]> * Fix linting * update caching policy for static content cdn * set cors configuration for static-contents * update cloufront cors headers with full domains * alarm for opennext lambda errors * config opennext instrumentation * config opennext instrumentation * instrumentation configs * fix linting * rollback confgurations to instrument next.js with telemetry * rollback package.json nextjs website * [DEV-2350] Add logs in guide pages (#1476) * Add logs to guide page fetch * add changeset * update iam policy to invalidate cloudfront opennext cache * update action deploy standalow: clean up configurations * [CAI-437] - Fix user feedback bug (#1459) * Fix user feedback bug * Add changeset * Add changeset * Remove double changeset --------- Co-authored-by: marcobottaro <[email protected]> * [CAI-445] - API documents in the index (#1466) * Update modules to read API docs and put them into the index * Update main prompt * Update modules and set sentence splitter as node parser * Update notebook * Update params for the sentence splitter node parser * Add changeset * Update documents.py to use strapi api key * Update chatbot.py * Update vector_database.py * Update engine.py * Update engine.py * Update models.py * Update creation index script * fixed typo * Update strapi env var from SSM * Update chatbot.py * Update documents.py * Update documents.py --------- Co-authored-by: marcobottaro <[email protected]> * [DEV-2344] Fix call order in fetch local docs (#1474) * Fix fetch-local-docs * Add changeset --------- Co-authored-by: marcobottaro <[email protected]> * rebase poc stanalone * Run standalone build in self hosted runner * [DEV-2347] Convert to static pages coming from gitbook (#1473) * add to static params page from gitbook * add changeset * Fix gitbook parsing for solutions and release notes * Fix release note pages * remove console log * fix generate metadata skip summary * Add changeset * Fix after review * [DEV-2342] Add generic nextjs error (#1486) * Add generic error * add changeset * Fix changeset --------- Co-authored-by: Marco Bottaro <[email protected]> Co-authored-by: christian-calabrese <[email protected]> Co-authored-by: tommaso1 <[email protected]> Co-authored-by: t <[email protected]> Co-authored-by: marcobottaro <[email protected]> Co-authored-by: uolter <[email protected]> Co-authored-by: Marcello Bertoli <[email protected]> Co-authored-by: MarBert <[email protected]> Co-authored-by: Marco Domenico Cirillo <[email protected]>
Sebastiano-Bertolin
pushed a commit
that referenced
this pull request
Jul 1, 2025
* Add logs to guide page fetch * add changeset
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
List of Changes
add logs for guide pages
Motivation and Context
logs are needed for performance measurement
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: