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

[FEATURE] Allow custom title instead file name #110

Closed
wants to merge 2 commits into from
Closed

[FEATURE] Allow custom title instead file name #110

wants to merge 2 commits into from

Conversation

afoeder
Copy link

@afoeder afoeder commented Aug 26, 2019

This introduces an opportunity to provide an optional query
parameter named documentTitle which will then be used instead
of the file name as title.

In order to achieve this some minor refactoring is done additionally.

Besides, it adjusts a travis configuration that seems to be necessary: Travis' Ubuntu has been upgraded to 16.4 where the package libconf is not included anymore, but required by Cypress in order to run the end to end tests. That change adds libgconf to the travis config explicitly. This has been mentioned at cypress-io/cypress#4069.

This adds the libgconf package as an addon
to Travis for getting Cypress to run. Apparently
travis upgraded to Ubuntu 16.4 where this package
is not intrinsically included anymore.

Find more about this at
cypress-io/cypress#4069

Besides, the TEST_URL environment variable is added
as a travis.yml directive in order to work across forks.
This introduces an opportunity to provide an optional query
parameter named documentTitle which will then be used instead
of the file name as title.

In order to achieve this some minor refactoring is done additionally.
@Vaishnavij
Copy link
Contributor

Hello @afoeder ,

Hope you are doing well!
The code changes in the PR, has diverged from current implementation. Is it possible to sync them with latest codebase?

Thanks,
BYOR Team

@Vaishnavij Vaishnavij closed this Apr 13, 2022
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