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

Update cms agilitycms #27378

Closed

Conversation

joshua-isaac
Copy link

Bug

  • Related issues linked using fixes #number
  • Integration tests added
  • Errors have helpful link attached, see contributing.md

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes #number
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.
  • Errors have helpful link attached, see contributing.md

Documentation / Examples

  • Make sure the linting passes

@ijjk ijjk added the examples Issue/PR related to examples label Jul 21, 2021
<Link href={`/blog/${featuredPost.fields.slug}`}>
<a className="cursor-pointer">
<div className="h-64 sm:h-96 relative">
<AgilityImage
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any insight why Agility image over next/image?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@leerob AgilityImage actually uses next/image behind the scenes! It avoids the need for any other config as it just changes the image URL parameters to resize it or change format and quality on our CDN

return dynamicUrls;
};

PostsListing.getCustomInitialProps = async ({
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, what is this?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@leerob getCustomInitialProps fetches additional data from Agility during the build process

examples/cms-agilitycms/next.config.js Outdated Show resolved Hide resolved
examples/cms-agilitycms/package.json Outdated Show resolved Hide resolved

export default async (req, res) => {

//TODO: Only generate the preview link if you are already in Preview!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO, also formatting 👍

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

created an issue so we have this todo tracked: agility/agilitycms-nextjs-starter#24

examples/cms-agilitycms/pages/api/hello.js Outdated Show resolved Hide resolved

let previewUrl = req.query.slug;

//TODO: these kinds of dynamic links should work by default (even outside of preview)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

created an issue so we have this todo tracked: agility/agilitycms-nextjs-starter#25

lineHeight: {
tight: 1.2,
teal: {
100: "#e6fffa",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this just use the default color palette from Tailwind?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I could just extend the default tailwind color palette instead of creating a new one, thanks!

@joshua-isaac joshua-isaac requested a review from leerob July 27, 2021 16:04
@balazsorban44
Copy link
Member

@joshua-isaac this PR has some merge conflicts and given it has been opened for a while, I'm closing it. If you would like to address the conflicts, feel free to reopen, otherwise, someone else might want to take a look! Thanks! 🙏 💚

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants