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

Nickname not present anymore #192

Open
plainerman opened this issue Nov 30, 2022 · 10 comments
Open

Nickname not present anymore #192

plainerman opened this issue Nov 30, 2022 · 10 comments
Labels
bug Related to a bug or a critical problem in the code. hacktoberfest This could be a good place to start if you want to contribute to this open source project. help request Issues raised by users who had problems running the script.

Comments

@plainerman
Copy link
Collaborator

According to #190, @JaapAntwerp posted:

The only thing still missing (which was there in the previous versions) is the "nickname" of the contact added to the body (as I have an extended family, sometimes in the "nickname" I have put information who is who. eg. "daughter of ...". or "son of .....")

If this was working in the past, we should reenable it.

@JaapAntwerp
Copy link

JaapAntwerp commented Dec 2, 2022

Could it be linked to this:

try {
self.contactId = googleContact.resourceName;
// Contact identification data.
self.data.merge(new ContactDataDC(
googleContact.names[0].displayName, // Name.
googleContact.givenName, // Nickname.
null // Profile image URL.
));

@plainerman
Copy link
Collaborator Author

Yes, it probably has to do with this line or at least it is a good starting point to debug.
If you (or anybody else) want(s), you can look into this issue and create a PR. I am here to answer questions.

If you don't want to I'll probably only find time over the holidays to look into this issue.

@plainerman plainerman added bug Related to a bug or a critical problem in the code. help request Issues raised by users who had problems running the script. labels Dec 6, 2022
@JaapAntwerp
Copy link

JaapAntwerp commented Dec 29, 2022 via email

@plainerman
Copy link
Collaborator Author

plainerman commented Dec 29, 2022

Hey @JaapAntwerp no worries, we are already grateful that you bring this to our attention.
But since I am currently the only maintainer and do this in my spare time I cannot guarantee you when I find the time to fix it. Be assured that it is on my to-do list ;).

I marked it as open if anybody else stumbles upon this issue and wants to fix it sooner than I'll find the time.

@plainerman plainerman added the hacktoberfest This could be a good place to start if you want to contribute to this open source project. label Dec 29, 2022
@JaapAntwerp
Copy link

JaapAntwerp commented Dec 31, 2022 via email

@plainerman
Copy link
Collaborator Author

I don't think that we will add support for this.

But the easiest way for you is to add one of these lines in the script

bodyPrefix = _('Hey! Don\'t forget these events') + ':';
bodySuffixes = [
_('Google Contacts Events Notifier') + ' (' + _('version') + ' ' + version.toString() + ')',
_('It looks like you are using an outdated version of this script') + '.',
_('You can find the latest one here')

For example you can change 'Google Contacts Events Notifier' to 'Google Contacts Events Notifier (Google Account1)'. If you use a different language, just change a string in your language settings.

For example if you use German use something here:

'de': {
'Age': 'Alter',
'Years': 'Jahre',
'Events': 'Termine',
'Birthdays today': 'Geburtstage heute',
'Birthdays tomorrow': 'Geburtstage morgen',
'Birthdays in {0} days': 'Geburtstage in {0} Tagen',
'Anniversaries today': 'Jahrestage heute',
'Anniversaries tomorrow': 'Jahrestage morgen',
'Anniversaries in {0} days': 'Jahrestage in {0} Tagen',
'Custom events today': 'Benutzerdefinierte Termine heute',
'Custom events tomorrow': 'Benutzerdefinierte Termine morgen',
'Custom events in {0} days': 'Benutzerdefinierte Termine in {0} Tagen',
'Hey! Don\'t forget these events': 'Hey! Vergiss diese Termine nicht',
'version': 'Version',
'dd-MM-yyyy': 'dd-MM-yyyy',
'Mobile phone': 'Telefon (mobil)',
'Work phone': 'Telefon (geschäftlich)',
'Home phone': 'Telefon (privat)',
'Main phone': 'Telefon (haupt)',
'Other phone': 'Telefon (sonstige)',
'Home fax': 'Fax (privat)',
'Work fax': 'Fax (geschäftlich)',
'Google voice': 'Google Voice',
'Pager': 'Pager',
'Home email': 'E-Mail (privat)',
'Work email': 'E-Mail (geschäftlich)',
'Other email': 'E-Mail (sonstige)',
'It looks like you are using an outdated version of this script': 'Du verwendest anscheinend eine veraltete Version dieses Skripts',
'You can find the latest one here': 'Du findest die neuste Version hier', // Using feminime version of 'latest', because it refers to 'version'. There's possibility it won't fit into diffrent context.
},

@JaapAntwerp
Copy link

JaapAntwerp commented Dec 31, 2022 via email

@JaapAntwerp
Copy link

JaapAntwerp commented Jan 4, 2023 via email

@JaapAntwerp
Copy link

JaapAntwerp commented Jan 4, 2023 via email

@JaapAntwerp
Copy link

I see 3 errors in the 5.1.4 version:

  1. the "nickname" is no longer presented in the body

  2. the "Anniversaries" are not shown in a separate group anymore, but in "custom events"

  3. the "notificationBlacklist" events are no longer filtered out, but shown anyway (the "notificationBlacklist" doesn't work anymore)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Related to a bug or a critical problem in the code. hacktoberfest This could be a good place to start if you want to contribute to this open source project. help request Issues raised by users who had problems running the script.
Projects
None yet
Development

No branches or pull requests

2 participants