-
Notifications
You must be signed in to change notification settings - Fork 30
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
adjust design of documentation #309
base: development
Are you sure you want to change the base?
Conversation
Reviewer's Guide by SourceryThis pull request focuses on updating the documentation's design to align with the eventyay brand. It includes changes to the theme, content, and links to ensure consistency with the eventyay ecosystem. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @r3yc0n1c - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please verify that all new links (blog.eventyay.com, Twitter, etc.) are working correctly before merging.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
docs/_templates/index.html
Outdated
</div> | ||
<div class="wy-sectionbox"> | ||
<div class="wy-icon"> | ||
<a href="https://twitter.com/eventyay" target="_blank"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, my bad. Please change it to x.com/eventyay
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
<div class="wy-sectionbox"> | ||
<div class="wy-icon"> | ||
<a href="https://twitter.com/eventyay" target="_blank"> | ||
<span class="fa fa-twitter fa-fw"></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also use x here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
|
||
# mkdir /var/venueless-data | ||
# chown -R 15371:15371 /var/venueless-data | ||
# mkdir /var/eventyay-video-data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be tested first.
# sudo -u postgres createuser -P venueless | ||
# sudo -u postgres createdb -O venueless venueless | ||
# sudo -u postgres createuser -P eventyay-video | ||
# sudo -u postgres createdb -O eventyay-video eventyay-video |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we probably need to verify this first.
@@ -96,23 +96,23 @@ Now restart redis-server:: | |||
Config file | |||
----------- | |||
|
|||
We now create a config directory and config file for venueless:: | |||
We now create a config directory and config file for eventyay-video:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm. all this needs to be verified first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also change red colored headings to eventyay-blue and black background on the left to the light-blue of eventyay.
The CNAME is written to the dev branch. On merge the CNAME should always be written to the gh-pages branch. |
please share the color palette/codes for this. I've updated the eventyay-blue but couldn't find the light-blue. For now, I've followed the eventyay-tickets scheme... let me know how it looks. |
fixed, I'll update the other workflows as well |
Resolves #305
Summary
Please use the same theme with the eventyay colors and logo as eventyay-tickets.
Then adjust the design of the documentation as follows:
Changed the name "venueless" to "eventyay" everywhere
Updated Welcome to "Welcome to the documentation of the eventyay tickets component!"
Used eventyay logo and color as in Django
Changed "Copyright Notice" to "© 2025 Apache 2.0 License by contributors" and reduced font size of Copyright notice
Changed blog to https://blog.eventyay.com/
Added eventyay favicon
Github https://github.com/fossasia/eventyay-video/
Twitter to X and link to https://twitter.com/eventyay
Changed let us know link https://github.com/fossasia/eventyay-video/issues/new
Changed other instances where links and mentioning to venuelss or other channels should be updated, e.g. also plugins and fix them.
Added custom footer layout
Dynamic date for Copyright
Added custom index file similar to eventyay-tickets
Update org email
Screenshot
Summary by Sourcery
Documentation: