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

🔥 Add Global ViewLayout #1384

Merged
merged 6 commits into from
Jun 14, 2021
Merged

🔥 Add Global ViewLayout #1384

merged 6 commits into from
Jun 14, 2021

Conversation

sujit-baniya
Copy link
Contributor

Please provide enough information so that others can review your pull request:
This PR will resolve #1382 by adding ViewLayout in fiber.Config allowing users to not define layout everywhere
Explain the details for making this change. What existing problem does the pull request solve?
This change allows user pass layout as global config and User don't need to pass layout on each render

@ReneWerner87
Copy link
Member

Thanks, can you please leave another description in the readme or in the docs repository

@sujit-baniya
Copy link
Contributor Author

@ReneWerner87 Added description on docs: gofiber/docs#179

@ReneWerner87
Copy link
Member

thx, mist forgot to mention the first time, can you please add another testcase

@ReneWerner87
Copy link
Member

@sujit-baniya
Copy link
Contributor Author

@ReneWerner87 It seems need to add most of the functionality for template to be added. Will it be acceptable if I add test template engine similar to html template for test?

@ReneWerner87
Copy link
Member

use the template test engine as in the other tests

@sujit-baniya
Copy link
Contributor Author

@ReneWerner87 Added test case for Views Layout

@ReneWerner87 ReneWerner87 changed the title 🔥 Add Global ViewLayout 🔥 Add Global ViewLayout Jun 14, 2021
@ReneWerner87 ReneWerner87 merged commit cb5f2f7 into gofiber:master Jun 14, 2021
@ReneWerner87 ReneWerner87 self-requested a review June 14, 2021 09:11
Copy link
Member

@ReneWerner87 ReneWerner87 left a comment

Choose a reason for hiding this comment

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

thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🚀 [FEATURE] - Add global template layout and global map for templates
2 participants