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

Account awaiting activation notification settings #1466

Closed
2 tasks done
Sama34 opened this issue Sep 21, 2014 · 12 comments · Fixed by #1710
Closed
2 tasks done

Account awaiting activation notification settings #1466

Sama34 opened this issue Sep 21, 2014 · 12 comments · Fixed by #1710
Assignees
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Milestone

Comments

@Sama34
Copy link
Contributor

Sama34 commented Sep 21, 2014

The notification bar should have some settings or be affected by current ones.

  • Turn on/off setting.
  • ACP link?

Reference: http://community.mybb.com/thread-158427.html

@Sama34 Sama34 added t:enhancement Type: Enhancement. Contains minor improvements b:1.8 Branch: 1.8.x s:feedback Status: Feedback. Further information/input needed p:medium Priority: Medium. Issues to be resolved with normal preference labels Sep 21, 2014
@Sama34 Sama34 self-assigned this Sep 21, 2014
@Sama34 Sama34 added this to the 1.8.1 milestone Sep 21, 2014
@DiogoParrinha DiogoParrinha modified the milestones: 1.8.2, 1.8.1 Oct 4, 2014
@Sama34 Sama34 added s:resolved Status: Resolved. Solution implemented or scheduled and removed s:feedback Status: Feedback. Further information/input needed labels Oct 11, 2014
@Sama34 Sama34 modified the milestones: 1.8.1, 1.8.2 Oct 11, 2014
@DiogoParrinha
Copy link
Contributor

@Sama34 Sama34 modified the milestones: 1.8.2, 1.8.1 Oct 14, 2014
@Stefan-MyBB
Copy link
Contributor

@Sama34 We can't merge your pull request.

@Eldenroot
Copy link
Contributor

@Sama34 - could you make a new PR?

In my opinion "Turn on/off setting" is a must have function, it would be nice to have it in 1.8.4 because many users complain about it in forum :(

@DiogoParrinha
Copy link
Contributor

@Sama34 your PR has dozens of changes. Any idea why? 99% don't affect this.

@Sama34
Copy link
Contributor Author

Sama34 commented Dec 19, 2014

I'm not sure, I can't fix it so as already mentioned in the PR I need #1389 to work on this and other issues.

@Sama34
Copy link
Contributor Author

Sama34 commented Dec 21, 2014

I added a new PR.

@Destroy666x
Copy link
Contributor

It fixes the 1st checkbox, what about the other 2?

@Sama34
Copy link
Contributor Author

Sama34 commented Dec 21, 2014

I can't recall why I didn't fixed those. I can't even recall what the acp link one is about, lol

@Destroy666x
Copy link
Contributor

It's about turning the notification (or a part of it) into a link to ACP page with a list of unactivated accounts:
admin/index.php?module=user-users&action=search&results=1&conditions=a%3A1%3A%7Bs%3A9%3A"usergroup"%3Bs%3A1%3A"5"%3B%7D
So a quite easy task.

Not sure if the 3rd checkbox is needed after an additional setting for the notification was added. I wouldn't change that, admins can simply turn off the bar if they use a different registration method or leave it on if they still want it.

@Sama34
Copy link
Contributor Author

Sama34 commented Dec 21, 2014

That link is quite different from the one shown in the ACP home page /index.php?module=user-users&action=search&results=1&conditions=a%3A1%3A%7Bs%3A9%3A%22usergroup%22%3Bs%3A1%3A%225%22%3B%7D&from=home. I have committed the change though.

@Destroy666x
Copy link
Contributor

I don't see any difference except the &from=home at the end, which only tells that you jumped to that page from the ACP index. And the " are encoded (should be encoded in the core file too and & changed to &).

@Sama34
Copy link
Contributor Author

Sama34 commented Dec 25, 2014

I moved the link to the language file since it is more accessible for administrators to edit it there (for whatever reason). I also encoded the &'s.

Destroy666x added a commit that referenced this issue Jan 17, 2015
Fix #1466 Account awaiting activation notification settings
@Destroy666x Destroy666x removed the p:medium Priority: Medium. Issues to be resolved with normal preference label Jan 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Projects
None yet
5 participants